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.

Sign up for GitHub

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

Jump to bottom

T ronmoneta/fix cloudinfo test #511

Merged
merged 5 commits into from
Nov 21, 2023
Merged

T ronmoneta/fix cloudinfo test #511

merged 5 commits into from
Nov 21, 2023

Conversation

t-ronmoneta
Copy link
Contributor

Added

Changed

  • changed redirect url to localhost per new url
  • skipped 404 e2e test as not all clusters changed yet, so failing atm

Fixed

@github-actions GitHub Actions
Copy link

github-actions bot commented Nov 21, 2023 •
edited
Loading

Test Results

       5 files  ±0         5 suites  ±0   18m 53s ⏱️ + 4m 36s
   293 tests ±0     273 ✔️ ±0    20 💤 +1  0 ❌  - 1 
1 465 runs  ±0  1 365 ✔️ ±0  100 💤 +5  0 ❌  - 5 

Results for commit 4fa610d. ± Comparison against base commit 86a6255.

This pull request skips 1 test. 
azure-kusto-data.tests.test_e2e_data.TestE2E ‑ test_cloud_info_404

♻️ This comment has been updated with latest results.

@t-ronmoneta t-ronmoneta merged commit 23a1d6c into master Nov 21, 2023
10 checks passed
@t-ronmoneta t-ronmoneta deleted the t-ronmoneta/fix_cloudinfo_test branch November 21, 2023 12:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AsafMah AsafMah AsafMah approved these changes

@yogilad yogilad Awaiting requested review from yogilad

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@t-ronmoneta @AsafMah