Skip to content

Commit

Permalink
Merge pull request #31624 from OlimpiaZurek/fix/31145-reassure-perf-t…
Browse files Browse the repository at this point in the history
…est-workflow

[NoQA] change ressure perf test workflow
  • Loading branch information
mountiny authored Nov 24, 2023
2 parents a2fc32b + cdc3424 commit e0f849f
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 3 deletions.
4 changes: 3 additions & 1 deletion .github/workflows/reassurePerformanceTests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,11 +22,13 @@ 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 --force --detach -
git switch "$CURRENT_BRANCH"
git merge --no-commit --no-ff "$BASELINE_BRANCH"
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.skip('should render ReportScreen with composer interactions', () => {
test('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.skip('should render ReportScreen with composer interactions', () => {
.then(() => measurePerformance(<ReportScreenWrapper route={mockRoute} />, {scenario, runs}));
});

test.skip('should press of the report item', () => {
test('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 e0f849f

Please sign in to comment.