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

Make hover e2e tests fixture messages more generic #934

Merged
merged 2 commits into from
Jun 28, 2023

Conversation

priyamsahoo
Copy link
Contributor

This PR makes hover e2e tests fixture messages more generic for the assertion to avoid failures because of markdown syntax.

@priyamsahoo priyamsahoo added the bug Something isn't working label Jun 28, 2023
@priyamsahoo priyamsahoo requested a review from a team as a code owner June 28, 2023 08:52
@priyamsahoo priyamsahoo force-pushed the fix-e2e-tests-after-antsibull branch from 23dd1c3 to 2279955 Compare June 28, 2023 09:27
@priyamsahoo priyamsahoo marked this pull request as draft June 28, 2023 09:28
@priyamsahoo priyamsahoo force-pushed the fix-e2e-tests-after-antsibull branch from 2279955 to 67887e3 Compare June 28, 2023 09:45
@priyamsahoo priyamsahoo force-pushed the fix-e2e-tests-after-antsibull branch from 67887e3 to 7f43316 Compare June 28, 2023 09:57
@priyamsahoo priyamsahoo marked this pull request as ready for review June 28, 2023 10:07
@priyamsahoo priyamsahoo merged commit 33567bd into ansible:main Jun 28, 2023
6 of 7 checks passed
@priyamsahoo priyamsahoo mentioned this pull request Jul 28, 2023
priyamsahoo added a commit that referenced this pull request Jul 28, 2023
## v2.5

### Minor Changes

- Upgrade als to v1.2.1 (#950) @priyamsahoo
- Add ability to select and set cursor on vars following `_var_` pattern (#884) @priyamsahoo
- Upgrade ALS to v1.1.0 (#932) @priyamsahoo
- Upgrade yaml package to 2.x (#925) @priyamsahoo

### Bugfixes

- Upgrade als to v1.2.1 (#950) @priyamsahoo
- Fix command runner (#949) @priyamsahoo
- Update lightspeed task name regex match condition  (#947) (#948) @ganeshrn
- Fix ansibleContent feedback event trigger (#940) @ganeshrn
- Update runner.ts to correctly get the path of ansible (#945) @priyamsahoo
- Make hover e2e tests fixture messages more generic (#934) @priyamsahoo


for more information, see https://pre-commit.ci

---------

Co-authored-by: Ansible DevTools <ansible-devtools@redhat.com>
Co-authored-by: Priyam Sahoo <42550351+priyamsahoo@users.noreply.github.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
github-actions bot pushed a commit that referenced this pull request Aug 1, 2023
## v2.6

### Minor Changes

- Upgrade als to v1.2.1 (#950) @priyamsahoo
- Add ability to select and set cursor on vars following `_var_` pattern (#884) @priyamsahoo
- Upgrade yaml package to 2.x (#925) @priyamsahoo

### Bugfixes

- Upgrade als to v1.2.1 (#950) @priyamsahoo
- Fix command runner (#949) @priyamsahoo
- Update lightspeed task name regex match condition  (#947) (#948) @ganeshrn
- Fix ansibleContent feedback event trigger (#940) @ganeshrn
- Update runner.ts to correctly get the path of ansible (#945) @priyamsahoo
- Make hover e2e tests fixture messages more generic (#934) @priyamsahoo
@ansibot ansibot mentioned this pull request Aug 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants