Skip to content

Tests cleanup - #490

Merged
rodlukas merged 1 commit into
masterfrom
tests-2026-10-20
Oct 28, 2026
Merged

rodlukas merged 1 commit into
masterfrom
tests-2026-10-20

Conversation

@rodlukas

Copy link
Copy Markdown
Owner

No description provided.

…rios, revert Applications refactor

- ui_environment: remove TESTS_CHROME_DEBUGGER_ADDRESS (attach to a running Chromium); keep
  TESTS_CHROME_BINARY, the keychain flags and the macOS check for the installed Google Chrome
- remove scenarios that repeat other coverage: client 404 (DRF default), edit/delete from search
  (same flow as the card), three API bank scenarios (covered by api/tests/test_bank.py)
- bank steps live in ui_steps/bank.py, api_steps/bank.py and common_steps/bank.py are gone
- flows.py imports the step modules through one helper instead of repeated use_step_matcher
- navigation step fails on the first section without the expected heading
- Applications.tsx back to the master version (refactor without a behavior change)
- tests/README and .env.template follow

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@rodlukas rodlukas changed the title Tests 2026 10 20 Tests cleanup Oct 28, 2026
@sonarqubecloud

Copy link
Copy Markdown

@rodlukas
rodlukas enabled auto-merge (squash) October 28, 2026 08:02
@codecov

codecov Bot commented Oct 28, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.06%. Comparing base (202fa0c) to head (8fa0188).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #490   +/-   ##
=======================================
  Coverage   97.06%   97.06%           
=======================================
  Files          22       22           
  Lines        1566     1566           
=======================================
  Hits         1520     1520           
  Misses         46       46           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@rodlukas
rodlukas merged commit 33ff17e into master Oct 28, 2026
15 checks passed
@rodlukas
rodlukas deleted the tests-2026-10-20 branch October 28, 2026 08:05

This branch was successfully deployed

1 active deployment
testing — 8fa0188b Deployed Oct 28, 2026 by rodlukas via Deploy to test Fly.io #1088
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.

1 participant