Skip to content

Commit

Permalink
ci: path resolving for windows grpc test
Browse files Browse the repository at this point in the history
  • Loading branch information
YOU54F committed Feb 24, 2024
1 parent b04deed commit 09d9755
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 7 deletions.
6 changes: 4 additions & 2 deletions .github/workflows/build-and-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -93,8 +93,10 @@ jobs:
strategy:
fail-fast: false
matrix:
node-version: [16,18,20]
os: [macos-14, macos-12, ubuntu-latest,windows-latest]
node-version: [20]
os: [macos-14, windows-latest]
# node-version: [16,18,20]
# os: [macos-14, macos-12, ubuntu-latest,windows-latest]

env:
NODE_VERSION: ${{ matrix.node-version }}
Expand Down
12 changes: 7 additions & 5 deletions test/message.integration.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -147,10 +147,12 @@ describe('FFI integration test for the Message Consumer API', () => {
});

describe('with plugin contents (gRPC)', () => {
const protoFile = path
.join(__dirname, 'integration', 'grpc', 'route_guide.proto')
.replace('\\\\', '/')
.replace('\\', '/');
const protoFile = path.join(
__dirname,
'integration',
'grpc',
'route_guide.proto'
);

let port: number;

Expand All @@ -160,7 +162,7 @@ describe('FFI integration test for the Message Consumer API', () => {

beforeEach(() => {
const grpcInteraction = `{
"pact:proto": "${protoFile}",
"pact:proto": "${protoFile.replace('\\\\', '/').replace('\\', '/')}",
"pact:proto-service": "RouteGuide/GetFeature",
"pact:content-type": "application/protobuf",
"request": {
Expand Down

0 comments on commit 09d9755

Please sign in to comment.