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

test: update /dev prepareRepoData unit test to include devfile filtering #5980

Conversation

willyyhuang
Copy link
Contributor

Problem

Follow up PR from #5853

Solution

Added more unit test coverage


License: I confirm that my contribution is made under the terms of the Apache 2.0 license.

@willyyhuang willyyhuang requested a review from a team as a code owner November 12, 2024 16:15
@willyyhuang willyyhuang force-pushed the wlsun/update-devfile-flow-test branch from 8d4bad4 to 42f0bb6 Compare November 12, 2024 17:08
@willyyhuang willyyhuang force-pushed the wlsun/update-devfile-flow-test branch from 42f0bb6 to 4563f1c Compare November 12, 2024 18:32
@willyyhuang willyyhuang requested a review from a team as a code owner November 12, 2024 18:32
@willyyhuang willyyhuang force-pushed the wlsun/update-devfile-flow-test branch from 4563f1c to 6c28506 Compare November 13, 2024 17:28
@willyyhuang willyyhuang force-pushed the wlsun/update-devfile-flow-test branch from 6c28506 to 996625c Compare November 14, 2024 15:18
@jpinkney-aws
Copy link
Contributor

Looks like should include devfile.yaml when setting is enabled is failing in ci

@willyyhuang
Copy link
Contributor Author

Looks like should include devfile.yaml when setting is enabled is failing in ci

taking a look

@willyyhuang
Copy link
Contributor Author

this was the error: TypeError: Attempted to wrap getDevCommandWorkspaceConfigurations which is already wrapped

will add a sinon.restore in afterEach hook to reset the mocks

@willyyhuang willyyhuang force-pushed the wlsun/update-devfile-flow-test branch from 996625c to 38771f8 Compare November 14, 2024 16:17
@willyyhuang willyyhuang force-pushed the wlsun/update-devfile-flow-test branch from 38771f8 to f3ecb62 Compare November 14, 2024 17:14
@jpinkney-aws jpinkney-aws merged commit ac76171 into aws:feature/q-dev-execution Nov 14, 2024
22 of 25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants