Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Integration tests failures #982

Closed
7 tasks
zFernand0 opened this issue Aug 17, 2020 · 2 comments
Closed
7 tasks

Integration tests failures #982

zFernand0 opened this issue Aug 17, 2020 · 2 comments
Labels
priority-low Legit issue but cosmetic or nice-to-have Technical Debt Includes Architecture, Code, Testing, Automation debt

Comments

@zFernand0
Copy link
Member

zFernand0 commented Aug 17, 2020

Describe the bug

After intermittent integration tests failures (#892) seem to have been resolved (setup and cleanup of tests).
There are still 3 tests failing (see full description in #929 (comment))

  • MVS command
  • Removing favorites
  • Deleting datasets (Verified user)

There are other tests that can fail depending on user environment. This are related to the use of Thenable in the test. For example:

  • DatasetTree Integration Tests: Tests that getChildren returns valid list of elements
  • ZoweNode Integration Tests: Testing that getChildren returns the correct Thenable<ZoweDatasetNode[]>

The last set of integrations test failures people can get are due to system configuration.

  • Recall tests (2 tests)
  • Migrate tests (2 tests)

To Reproduce

Run integration tests

Expected behavior

All Integration tests passing! 😋

Screenshots

image
Credits: #787 (comment)

Desktop (please complete the following information):

Additional context

None

@zFernand0 zFernand0 added the Technical Debt Includes Architecture, Code, Testing, Automation debt label Aug 17, 2020
@lauren-li
Copy link
Contributor

I am not actively working on this issue, but I have incidentally fixed the failing removeFavorite() integration test as part of my PR #984 ("Issue #968: Group data sets’ favorites by profile and allow extenders to activate before loading profiles").

@JillieBeanSim JillieBeanSim added the priority-low Legit issue but cosmetic or nice-to-have label Jan 25, 2023
@zFernand0
Copy link
Member Author

closing in favor of:

@zFernand0 zFernand0 closed this as not planned Won't fix, can't repro, duplicate, stale Feb 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority-low Legit issue but cosmetic or nice-to-have Technical Debt Includes Architecture, Code, Testing, Automation debt
Projects
None yet
Development

No branches or pull requests

3 participants