Skip to content

Commit

Permalink
Merge pull request #31825 from OlimpiaZurek/revert-31624-fix/31145-re…
Browse files Browse the repository at this point in the history
…assure-perf-test-workflow

Revert "[NoQA] change ressure perf test workflow"
  • Loading branch information
mountiny authored Nov 24, 2023
2 parents 3ce5a89 + b99cc25 commit 19ab14a
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 5 deletions.
4 changes: 1 addition & 3 deletions .github/workflows/reassurePerformanceTests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,13 +22,11 @@ jobs:
run: |
set -e
BASELINE_BRANCH=${BASELINE_BRANCH:="main"}
CURRENT_BRANCH=$(git branch --show-current)
git fetch origin "$BASELINE_BRANCH" --no-tags --depth=1
git switch "$BASELINE_BRANCH"
npm install --force
npx reassure --baseline
git switch "$CURRENT_BRANCH"
git merge --no-commit --no-ff "$BASELINE_BRANCH"
git switch --force --detach -
npm install --force
npx reassure --branch
Expand Down
4 changes: 2 additions & 2 deletions tests/perf-test/ReportScreen.perf-test.js
Original file line number Diff line number Diff line change
Expand Up @@ -124,7 +124,7 @@ function ReportScreenWrapper(args) {

const runs = CONST.PERFORMANCE_TESTS.RUNS;

test('should render ReportScreen with composer interactions', () => {
test.skip('should render ReportScreen with composer interactions', () => {
const scenario = async () => {
// Query for the report list
await screen.findByTestId('report-actions-list');
Expand Down Expand Up @@ -175,7 +175,7 @@ test('should render ReportScreen with composer interactions', () => {
.then(() => measurePerformance(<ReportScreenWrapper route={mockRoute} />, {scenario, runs}));
});

test('should press of the report item', () => {
test.skip('should press of the report item', () => {
const scenario = async () => {
// Query for the report list
await screen.findByTestId('report-actions-list');
Expand Down

0 comments on commit 19ab14a

Please sign in to comment.