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

Disable LayerExportControllerIntegrationTest#shouldNotExportHiddenAttributesInGeoJSON #935

Merged
merged 3 commits into from
Sep 2, 2024

Conversation

mprins
Copy link
Contributor

@mprins mprins commented Sep 2, 2024

as tracked in https://b3partners.atlassian.net/browse/HTM-1198 this may be caused by a behavioural change in GeoServer

@mprins mprins added the tests CI and build proces related or testcases label Sep 2, 2024
@mprins mprins self-assigned this Sep 2, 2024
Copy link

github-actions bot commented Sep 2, 2024

Test Results

266 tests  ±0   264 ✅ ±0   3m 17s ⏱️ +26s
 27 suites ±0     2 💤 +1 
 27 files   ±0     0 ❌  - 1 

Results for commit 596b689. ± Comparison against base commit 500c620.

This pull request skips 1 test.
org.tailormap.api.controller.LayerExportControllerIntegrationTest ‑ shouldNotExportHiddenAttributesInGeoJSON

♻️ This comment has been updated with latest results.

Copy link

codecov bot commented Sep 2, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Impacted file tree graph

@@           Coverage Diff           @@
##             main   #935     +/-   ##
=======================================
+ Coverage       5%    78%    +74%     
  Complexity     78     78             
=======================================
  Files          95     95             
  Lines        4499   4499             
  Branches      423    423             
=======================================
+ Hits          199   3487   +3288     
+ Misses       4281    788   -3493     
- Partials       19    224    +205     

see 88 files with indirect coverage changes

@mprins mprins merged commit 6fa9ab0 into main Sep 2, 2024
23 checks passed
@mprins mprins deleted the mprins-patch-1 branch September 2, 2024 07:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tests CI and build proces related or testcases
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant