Skip to content

Commit

Permalink
Disable subscription tests on Ubuntu
Browse files Browse the repository at this point in the history
  • Loading branch information
ThisaruGuruge committed Mar 6, 2025
1 parent caadbf1 commit 8b46aa8
Show file tree
Hide file tree
Showing 10 changed files with 94 additions and 53 deletions.
2 changes: 0 additions & 2 deletions .github/workflows/build-timestamped-master.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,5 +17,3 @@ jobs:
if: ${{ github.repository_owner == 'ballerina-platform' }}
uses: ballerina-platform/ballerina-library/.github/workflows/build-timestamp-master-template.yml@2201.10.x
secrets: inherit
with:
additional-build-flags: "-P disable=subscriptions"
3 changes: 1 addition & 2 deletions .github/workflows/publish-release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,5 +14,4 @@ jobs:
with:
package-name: graphql
package-org: ballerina
additional-build-flags: '-x :graphql-examples:build -P disable=subscriptions'
additional-publish-flags: "-P disable=subscriptions"
additional-build-flags: '-x :graphql-examples:build'
2 changes: 0 additions & 2 deletions .github/workflows/pull-request.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,5 +12,3 @@ jobs:
if: ${{ github.repository_owner == 'ballerina-platform' }}
uses: ballerina-platform/ballerina-library/.github/workflows/pull-request-build-template.yml@2201.10.x
secrets: inherit
with:
additional-ubuntu-test-flags: "-P disable=subscriptions"
3 changes: 2 additions & 1 deletion ballerina-tests/tests/01_listener_test.bal
Original file line number Diff line number Diff line change
Expand Up @@ -124,7 +124,8 @@ function testAttachServiceWithMutationToHttp1BasedListenerAndClient() returns er
}

@test:Config {
groups: ["listener", "subscriptions"]
groups: ["listener", "subscriptions"],
enable: false
}
function testAttachServiceWithSubscriptionToHttp1BasedListener() returns error? {
string document = string `subscription { messages }`;
Expand Down
18 changes: 12 additions & 6 deletions ballerina-tests/tests/02_request_validation.bal
Original file line number Diff line number Diff line change
Expand Up @@ -147,7 +147,8 @@ isolated function testInvalidRequestBody() returns error? {
}

@test:Config {
groups: ["request_validation", "websocket", "subscriptions"]
groups: ["request_validation", "websocket", "subscriptions"],
enable: false
}
isolated function testInvalidWebSocketRequestWithEmptyQuery() returns error? {
string document = "";
Expand All @@ -161,7 +162,8 @@ isolated function testInvalidWebSocketRequestWithEmptyQuery() returns error? {
}

@test:Config {
groups: ["request_validation", "websocket", "subscriptions"]
groups: ["request_validation", "websocket", "subscriptions"],
enable: false
}
isolated function testInvalidWebSocketRequestWithInvalidQuery() returns error? {
string url = "ws://localhost:9099/subscriptions";
Expand All @@ -176,7 +178,8 @@ isolated function testInvalidWebSocketRequestWithInvalidQuery() returns error? {
}

@test:Config {
groups: ["request_validation", "websocket", "subscriptions"]
groups: ["request_validation", "websocket", "subscriptions"],
enable: false
}
isolated function testInvalidWebSocketRequestWithoutQuery() returns error? {
string url = "ws://localhost:9099/subscriptions";
Expand All @@ -190,7 +193,8 @@ isolated function testInvalidWebSocketRequestWithoutQuery() returns error? {
}

@test:Config {
groups: ["request_validation", "websocket", "subscriptions"]
groups: ["request_validation", "websocket", "subscriptions"],
enable: false
}
isolated function testInvalidVariableInWebSocketPayload() returns error? {
string document = check getGraphqlDocumentFromFile("subscriptions_with_variable_values");
Expand All @@ -206,7 +210,8 @@ isolated function testInvalidVariableInWebSocketPayload() returns error? {
}

@test:Config {
groups: ["request_validation", "websocket", "subscriptions"]
groups: ["request_validation", "websocket", "subscriptions"],
enable: false
}
isolated function testEmptyWebSocketPayload() returns error? {
string url = "ws://localhost:9099/subscriptions";
Expand All @@ -220,7 +225,8 @@ isolated function testEmptyWebSocketPayload() returns error? {
}

@test:Config {
groups: ["request_validation", "websocket", "subscriptions"]
groups: ["request_validation", "websocket", "subscriptions"],
enable: false
}
isolated function testInvalidWebSocketPayload() returns error? {
string url = "ws://localhost:9099/subscriptions";
Expand Down
3 changes: 2 additions & 1 deletion ballerina-tests/tests/27_context.bal
Original file line number Diff line number Diff line change
Expand Up @@ -317,7 +317,8 @@ isolated function testContextWithSubscriptions() returns error? {
}

@test:Config {
groups: ["context", "subscriptions"]
groups: ["context", "subscriptions"],
enable: false
}
isolated function testContextWithInvalidScopeInSubscriptions() returns error? {
string url = "ws://localhost:9092/context";
Expand Down
78 changes: 52 additions & 26 deletions ballerina-tests/tests/36_subscriptions.bal
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,8 @@ import ballerina/test;
import ballerina/websocket;

@test:Config {
groups: ["subscriptions"]
groups: ["subscriptions"],
enable: false
}
isolated function testSubscription() returns error? {
string document = string `subscription { name }`;
Expand All @@ -35,7 +36,8 @@ isolated function testSubscription() returns error? {
}

@test:Config {
groups: ["subscriptions"]
groups: ["subscriptions"],
enable: false
}
isolated function testSubscriptionWithoutSubProtocol() returns error? {
string url = "ws://localhost:9099/subscriptions";
Expand All @@ -46,7 +48,8 @@ isolated function testSubscriptionWithoutSubProtocol() returns error? {
}

@test:Config {
groups: ["subscriptions"]
groups: ["subscriptions"],
enable: false
}
isolated function testSubscriptionsWithMultipleOperations() returns error? {
string document = check getGraphqlDocumentFromFile("subscriptions_with_multiple_operations");
Expand Down Expand Up @@ -75,7 +78,8 @@ isolated function testSubscriptionsWithMultipleOperations() returns error? {
}

@test:Config {
groups: ["records", "subscriptions"]
groups: ["records", "subscriptions"],
enable: false
}
isolated function testSubscriptionWithRecords() returns error? {
string document = check getGraphqlDocumentFromFile("subscriptions_with_records");
Expand All @@ -92,7 +96,8 @@ isolated function testSubscriptionWithRecords() returns error? {
}

@test:Config {
groups: ["subscriptions"]
groups: ["subscriptions"],
enable: false
}
isolated function testQueryWithSameSubscriptionFieldName() returns error? {
string document = string `query { name }`;
Expand All @@ -103,7 +108,8 @@ isolated function testQueryWithSameSubscriptionFieldName() returns error? {
}

@test:Config {
groups: ["fragments", "subscriptions"]
groups: ["fragments", "subscriptions"],
enable: false
}
isolated function testSubscriptionWithFragments() returns error? {
string document = check getGraphqlDocumentFromFile("subscriptions_with_fragments");
Expand All @@ -120,7 +126,8 @@ isolated function testSubscriptionWithFragments() returns error? {
}

@test:Config {
groups: ["union", "subscriptions"]
groups: ["union", "subscriptions"],
enable: false
}
isolated function testSubscriptionWithUnionType() returns error? {
string document = check getGraphqlDocumentFromFile("subscriptions_with_union_type");
Expand All @@ -137,7 +144,8 @@ isolated function testSubscriptionWithUnionType() returns error? {
}

@test:Config {
groups: ["variables", "subscriptions"]
groups: ["variables", "subscriptions"],
enable: false
}
isolated function testSubscriptionWithVariables() returns error? {
string document = check getGraphqlDocumentFromFile("subscriptions_with_variable_values");
Expand All @@ -155,7 +163,8 @@ isolated function testSubscriptionWithVariables() returns error? {
}

@test:Config {
groups: ["introspection", "typename", "subscriptions"]
groups: ["introspection", "typename", "subscriptions"],
enable: false
}
isolated function testSubscriptionWithIntrospectionInFields() returns error? {
string document = string `subscription { students { __typename } }`;
Expand All @@ -170,7 +179,8 @@ isolated function testSubscriptionWithIntrospectionInFields() returns error? {
}

@test:Config {
groups: ["subscriptions"]
groups: ["subscriptions"],
enable: false
}
isolated function testInvalidSubscription() returns error? {
string document = string `subscription { invalidField }`;
Expand All @@ -185,7 +195,8 @@ isolated function testInvalidSubscription() returns error? {
}

@test:Config {
groups: ["subscriptions"]
groups: ["subscriptions"],
enable: false
}
isolated function testSubscriptionFunctionWithErrors() returns error? {
string document = string `subscription getNames { values }`;
Expand All @@ -206,7 +217,8 @@ isolated function testSubscriptionFunctionWithErrors() returns error? {
}

@test:Config {
groups: ["service", "subscriptions"]
groups: ["service", "subscriptions"],
enable: false
}
isolated function testSubscriptionWithServiceObjects() returns error? {
string document = check getGraphqlDocumentFromFile("subscriptions_with_service_objects");
Expand All @@ -224,7 +236,8 @@ isolated function testSubscriptionWithServiceObjects() returns error? {
}

@test:Config {
groups: ["subscriptions"]
groups: ["subscriptions"],
enable: false
}
isolated function testSubscriptionWithMultipleClients() returns error? {
string document = string `subscription { messages }`;
Expand All @@ -249,7 +262,8 @@ isolated function testSubscriptionWithMultipleClients() returns error? {
}

@test:Config {
groups: ["subscriptions"]
groups: ["subscriptions"],
enable: false
}
isolated function testConnectionInitMessage() returns error? {
string url = "ws://localhost:9099/subscriptions";
Expand All @@ -260,7 +274,8 @@ isolated function testConnectionInitMessage() returns error? {
}

@test:Config {
groups: ["subscriptions"]
groups: ["subscriptions"],
enable: false
}
isolated function testInvalidMultipleConnectionInitMessages() returns error? {
string url = "ws://localhost:9099/subscriptions";
Expand All @@ -274,7 +289,8 @@ isolated function testInvalidMultipleConnectionInitMessages() returns error? {
}

@test:Config {
groups: ["subscriptions"]
groups: ["subscriptions"],
enable: false
}
isolated function testUnauthorizedAccess() returns error? {
string document = check getGraphqlDocumentFromFile("subscriptions_with_service_objects");
Expand All @@ -288,7 +304,8 @@ isolated function testUnauthorizedAccess() returns error? {
}

@test:Config {
groups: ["subscriptions"]
groups: ["subscriptions"],
enable: false
}
function testAlreadyExistingSubscriber() returns error? {
string document = check getGraphqlDocumentFromFile("subscriptions_with_service_objects");
Expand Down Expand Up @@ -320,7 +337,8 @@ function testAlreadyExistingSubscriber() returns error? {
}

@test:Config {
groups: ["subscriptions"]
groups: ["subscriptions"],
enable: false
}
isolated function testOnPing() returns error? {
string url = "ws://localhost:9099/subscriptions";
Expand All @@ -332,7 +350,8 @@ isolated function testOnPing() returns error? {
}

@test:Config {
groups: ["subscriptions"]
groups: ["subscriptions"],
enable: false
}
isolated function testInvalidSubProtocolInSubscriptions() returns error? {
string url = "ws://localhost:9099/subscriptions";
Expand All @@ -346,7 +365,8 @@ isolated function testInvalidSubProtocolInSubscriptions() returns error? {
}

@test:Config {
groups: ["subscriptions", "runtime_errors"]
groups: ["subscriptions", "runtime_errors"],
enable: false
}
isolated function testErrorsInStreams() returns error? {
string document = "subscription { evenNumber }";
Expand All @@ -365,7 +385,8 @@ isolated function testErrorsInStreams() returns error? {
}

@test:Config {
groups: ["subscriptions"]
groups: ["subscriptions"],
enable: false
}
isolated function testMultipleSubscriptionUsingSingleClient() returns error? {
string document = string `subscription { messages }`;
Expand All @@ -390,7 +411,8 @@ isolated function testMultipleSubscriptionUsingSingleClient() returns error? {
}

@test:Config {
groups: ["subscriptions"]
groups: ["subscriptions"],
enable: false
}
isolated function testSubscriptionWithInvalidPayload() returns error? {
string url = "ws://localhost:9099/subscriptions";
Expand All @@ -406,7 +428,8 @@ isolated function testSubscriptionWithInvalidPayload() returns error? {
}

@test:Config {
groups: ["subscriptions", "recrods", "service"]
groups: ["subscriptions", "records", "service"],
enable: false
}
isolated function testResolverReturingStreamOfRecordsWithServiceObjects() returns error? {
string document = "subscription { live { product { id } score } }";
Expand All @@ -421,7 +444,8 @@ isolated function testResolverReturingStreamOfRecordsWithServiceObjects() return
}

@test:Config {
groups: ["subscriptions", "recrods", "service", "maps"]
groups: ["subscriptions", "records", "service", "maps"],
enable: false
}
isolated function testResolverReturingStreamOfRecordsWithMapOfServiceObjects() returns error? {
string document = string `subscription { accountUpdates { details(key: "acc1") { name } } }`;
Expand All @@ -438,7 +462,8 @@ isolated function testResolverReturingStreamOfRecordsWithMapOfServiceObjects() r
}

@test:Config {
groups: ["subscriptions", "multiplexing"]
groups: ["subscriptions", "multiplexing"],
enable: false
}
isolated function testSubscriptionMultiplexing() returns error? {
string document = string `subscription { refresh }`;
Expand Down Expand Up @@ -473,7 +498,8 @@ isolated function testSubscriptionMultiplexing() returns error? {
}

@test:Config {
groups: ["subscriptions", "recrods", "service"]
groups: ["subscriptions", "records", "service"],
enable: false
}
isolated function testConnectionClousureWhenPongNotRecived() returns error? {
string url = "ws://localhost:9090/reviews";
Expand Down
Loading

0 comments on commit 8b46aa8

Please sign in to comment.