Skip to content

Commit d6351a8

Browse files
committed
Disable miq_debug for cypress for now
This should allow us to remove a lot of the force: true we had to add for many DOM elements near the top of the page as the notifications could cover these elements and various cypress interactions with covered elements would fail. Note, we'll need to figure out how to test notifications differently.
1 parent 1bd9f0f commit d6351a8

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed

.github/workflows/cypress.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,7 @@ jobs:
3535
- 11211:11211
3636
env:
3737
TEST_SUITE: spec:cypress
38+
CYPRESS: true
3839
CYPRESS_BROWSER: ${{ matrix.cypress-browser }}
3940
PGHOST: localhost
4041
PGPASSWORD: smartvm

app/javascript/oldjs/application.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ require('./miq_toolbar.js');
3131
require('./miq_c3.js');
3232
require('./miq_explorer.js');
3333

34-
if (process.env.NODE_ENV === 'development') {
34+
if (process.env.NODE_ENV === 'development' && process.env.CYPRESS !== 'true') {
3535
require('./miq_debug.js');
3636
require('./miq_debug.css');
3737
}

0 commit comments

Comments
 (0)