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

[MAINTENANCE] Clean up xfails in preparation for 0.15.20 release #5835

Merged
merged 1 commit into from Aug 25, 2022

Conversation

cdkini
Copy link
Member

@cdkini cdkini commented Aug 25, 2022

Changes proposed in this pull request:

  • Certain tests started passing this morning due to a change made in the Cloud backend; since we set strict, we need to modify these tests before the release.

Definition of Done

Please delete options that are not relevant.

@netlify
Copy link

netlify bot commented Aug 25, 2022

Deploy Preview for niobium-lead-7998 ready!

Name Link
🔨 Latest commit 160e888
🔍 Latest deploy log https://app.netlify.com/sites/niobium-lead-7998/deploys/63076f5d805a0b0009df018e
😎 Deploy Preview https://deploy-preview-5835--niobium-lead-7998.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@ghost
Copy link

ghost commented Aug 25, 2022

👇 Click on the image for a new way to code review
  • Make big changes easier — review code in small groups of related files

  • Know where to start — see the whole change at a glance

  • Take a code tour — explore the change with an interactive tour

  • Make comments and review — all fully sync’ed with github

    Try it now!

Review these changes using an interactive CodeSee Map

Legend

CodeSee Map Legend

@cdkini cdkini self-assigned this Aug 25, 2022
@cdkini cdkini enabled auto-merge (squash) August 25, 2022 12:49
@@ -252,7 +251,7 @@ def _closure(context: DataContext) -> Tuple[Validator, str]:


@pytest.mark.xfail(
reason="GX Cloud E2E tests are currently failing due to a schema issue with DataContextVariables; xfailing for purposes of the 0.15.20 release",
reason="GX Cloud E2E tests are currently failing due to an id issue with ExpectationSuites; xfailing for purposes of the 0.15.20 release",
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Separate issue I'll need to look into

@cdkini cdkini merged commit 2299992 into develop Aug 25, 2022
@cdkini cdkini deleted the maintenance/clean-up-xfails branch August 25, 2022 13:10
Shinnnyshinshin pushed a commit that referenced this pull request Aug 25, 2022
…ssed-in-as-batchspec-passthrough-spark

* develop:
  [MAINTENANCE] Add entries to release schedule
  [MAINTENANCE] Unit tests to ensure coverage over `Datasource` caching in `DataContext` (#5839)
  [MAINTENANCE] Unit Tests Pipeline step (#5838)
  [RELEASE] 0.15.20 (#5831)
  [MAINTENANCE] Add back xfails for E2E Cloud tests that fail on env var retrieval in Docker (#5837)
  [MAINTENANCE] Clean up xfails in preparation for 0.15.20 release (#5835)
Shinnnyshinshin pushed a commit that referenced this pull request Aug 26, 2022
…r-usage-stats-opt

* develop: (337 commits)
  [MAINTENANCE] Finer-grain type exclude (#5848)
  remove pyparsing pin for <3.0 (#5849)
  [MAINTENANCE] add --ignore-suppress and --ignore-only-for to build_gallery.py with bugfixes (#5802)
  [MAINTENANCE] `datasource`, `data_context`, `core` typing, lint fixes (#5824)
  [MAINTENANCE] Mark miscellaneous tests with `@pytest.mark.unit` (#5846)
  [MAINTENANCE] Add additional unit tests around `Datasource` caching (#5844)
  [MAINTENANCE] Properly label DataAssistant tests with @pytest.mark.integration
  [DOCS] DOC-364 how to configure a spark datasource (#5840)
  [MAINTENANCE] Add entries to release schedule
  [MAINTENANCE] Unit tests to ensure coverage over `Datasource` caching in `DataContext` (#5839)
  [MAINTENANCE] Unit Tests Pipeline step (#5838)
  [RELEASE] 0.15.20 (#5831)
  [MAINTENANCE] Add back xfails for E2E Cloud tests that fail on env var retrieval in Docker (#5837)
  [MAINTENANCE] Clean up xfails in preparation for 0.15.20 release (#5835)
  [MAINTENANCE] xfail Cloud E2E tests due to schema issue with `DataContextVariables` (#5828)
  [FEATURE] DataAssistants Example Notebook - Pandas (#5820)
  [MAINTENANCE] Update `list_` methods on `DataContext` to emit names along with object ids (#5826)
  Bugfix/snowflake temp table schema name (#5814)
  [FEATURE] Better error for missing expectation (#5750)
  [Maintenance] Clean up build_parameter_container_for_variables (#5823)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants