Skip to content

Commit

Permalink
yml update2 (#88)
Browse files Browse the repository at this point in the history
pre-install global package spfx-fast-serve in node 18 (any os) as a workaround

removed sortPackage tmp logging
  • Loading branch information
BassemNKhalil authored Nov 17, 2023
1 parent f64ffd8 commit 33bd738
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 5 deletions.
3 changes: 2 additions & 1 deletion .github/workflows/dispatch.yml
Original file line number Diff line number Diff line change
Expand Up @@ -24,5 +24,6 @@ jobs:
- uses: actions/setup-node@v3
with:
node-version: ${{ matrix.version }}
- run: npm config get cache
- run: npm i -g spfx-fast-serve
if: ${{ matrix.version == 18 }}
- run: npm cit
5 changes: 3 additions & 2 deletions .github/workflows/pull_request.yml
Original file line number Diff line number Diff line change
Expand Up @@ -25,8 +25,9 @@ jobs:
with:
node-version: ${{ matrix.version }}
cache: "npm"
- run: npm ci
- run: npm test
- run: npm i -g spfx-fast-serve
if: ${{ matrix.version == 18 }}
- run: npm cit
coverage-report:
uses: ./.github/workflows/jest_coverage_report.yml
needs: run-tests
Expand Down
2 changes: 0 additions & 2 deletions Tests/e2e/Commands/sortPackage.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -8,10 +8,8 @@ const packageJSON_filePath = join(wpFolderPath, "package.json");

beforeAll(() => {
packageJSON_beforeChanges = fs.readFileSync(packageJSON_filePath, "utf8");
console.log(`packageJSON_beforeChanges: ${packageJSON_beforeChanges}`);
sortPackageCommand.sortPackage(packageJSON_filePath);
packageJSON_afterChanges = fs.readFileSync(packageJSON_filePath, "utf8");
console.log(`packageJSON_afterChanges: ${packageJSON_afterChanges}`);
});

test("sortPackage: Package.json file changed", () => {
Expand Down

1 comment on commit 33bd738

@github-actions
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Coverage report

St.
Category Percentage Covered / Total
🟢 Statements 100% 124/124
🟢 Branches 100% 10/10
🟢 Functions 100% 13/13
🟢 Lines 100% 124/124

Test suite run success

26 tests passing in 11 suites.

Report generated by 🧪jest coverage report action from 33bd738

Please sign in to comment.