Skip to content

update loading test to best-practices#48429

Merged
vtjnash merged 1 commit intomasterfrom
jn/test-loading-best
Jan 31, 2023
Merged

update loading test to best-practices#48429
vtjnash merged 1 commit intomasterfrom
jn/test-loading-best

Conversation

@vtjnash
Copy link
Copy Markdown
Member

@vtjnash vtjnash commented Jan 27, 2023

Many of these test were hiding output or using semi-deprecated functions or failing to cleanup output or setting the wrong variables.

Many of these test were hiding output or using semi-deprecated functions
or failing to cleanup output or setting the wrong variables.
@vtjnash vtjnash added the test This change adds or pertains to unit tests label Jan 27, 2023
@vtjnash vtjnash merged commit b48104d into master Jan 31, 2023
@vtjnash vtjnash deleted the jn/test-loading-best branch January 31, 2023 19:01
@KristofferC KristofferC added the backport 1.9 Change should be backported to release-1.9 label Feb 20, 2023
KristofferC pushed a commit that referenced this pull request Feb 20, 2023
Many of these test were hiding output or using semi-deprecated functions
or failing to cleanup output or setting the wrong variables.

(cherry picked from commit b48104d)
KristofferC pushed a commit that referenced this pull request Feb 20, 2023
Many of these test were hiding output or using semi-deprecated functions
or failing to cleanup output or setting the wrong variables.

(cherry picked from commit b48104d)
KristofferC pushed a commit that referenced this pull request Feb 21, 2023
Many of these test were hiding output or using semi-deprecated functions
or failing to cleanup output or setting the wrong variables.

(cherry picked from commit b48104d)
KristofferC pushed a commit that referenced this pull request Feb 21, 2023
Many of these test were hiding output or using semi-deprecated functions
or failing to cleanup output or setting the wrong variables.

(cherry picked from commit b48104d)
KristofferC pushed a commit that referenced this pull request Feb 21, 2023
Many of these test were hiding output or using semi-deprecated functions
or failing to cleanup output or setting the wrong variables.

(cherry picked from commit b48104d)
@KristofferC KristofferC removed the backport 1.9 Change should be backported to release-1.9 label Mar 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test This change adds or pertains to unit tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants