diff --git a/.github/workflows/RavenClient.yml b/.github/workflows/RavenClient.yml index 2b6d2b6a..a9c569f4 100644 --- a/.github/workflows/RavenClient.yml +++ b/.github/workflows/RavenClient.yml @@ -12,7 +12,7 @@ on: ravendb_version: description: 'RavenDB Version' required: true - type: string + type: string jobs: build: @@ -27,6 +27,7 @@ jobs: RAVENDB_TEST_HTTPS_SERVER_URL: https://localhost:8989 RAVENDB_BUILD_TYPE: nightly RAVEN_License: ${{ secrets.RAVEN_LICENSE }} + RAVENDB_SERVER_VERSION: ${{ matrix.serverVersion }} strategy: matrix: diff --git a/src/Documents/Operations/Replication/PullReplicationAsSink.ts b/src/Documents/Operations/Replication/PullReplicationAsSink.ts index 14fc267f..9870d88a 100644 --- a/src/Documents/Operations/Replication/PullReplicationAsSink.ts +++ b/src/Documents/Operations/Replication/PullReplicationAsSink.ts @@ -10,5 +10,8 @@ export interface PullReplicationAsSink extends ExternalReplicationBase { accessName?: string; hubName?: string; + /** + * @deprecated Use HubName instead + */ hubDefinitionName?: string; -} \ No newline at end of file +} diff --git a/test/Ported/Graph/BasicGraphQueriesTest.ts b/test/Ported/Graph/BasicGraphQueriesTest.ts index 2bc459e8..348885db 100644 --- a/test/Ported/Graph/BasicGraphQueriesTest.ts +++ b/test/Ported/Graph/BasicGraphQueriesTest.ts @@ -1,11 +1,11 @@ import { IDocumentStore } from "../../../src/Documents/IDocumentStore"; -import { disposeTestDocumentStore, testContext } from "../../Utils/TestUtil"; +import { disposeTestDocumentStore, RavenTestContext, testContext } from "../../Utils/TestUtil"; import { DocumentType } from "../../../src/Documents/DocumentAbstractions"; import { CreateSampleDataOperation } from "../../Utils/CreateSampleDataOperation"; import { assertThat } from "../../Utils/AssertExtensions"; import { Movie } from "../../Assets/Graph"; -describe("BasicGraphQueriesTest", function () { +(RavenTestContext.is60Server ? describe.skip : describe)("BasicGraphQueriesTest", function () { let store: IDocumentStore; diff --git a/test/Ported/Graph/ClientGraphQueriesTest.ts b/test/Ported/Graph/ClientGraphQueriesTest.ts index 111af46b..0c8772d3 100644 --- a/test/Ported/Graph/ClientGraphQueriesTest.ts +++ b/test/Ported/Graph/ClientGraphQueriesTest.ts @@ -1,10 +1,10 @@ import { IDocumentStore } from "../../../src/Documents/IDocumentStore"; -import { disposeTestDocumentStore, testContext } from "../../Utils/TestUtil"; +import { disposeTestDocumentStore, RavenTestContext, testContext } from "../../Utils/TestUtil"; import { assertThat } from "../../Utils/AssertExtensions"; import moment = require("moment"); import { DateUtil } from "../../../src/Utility/DateUtil"; -describe("ClientGraphQueriesTest", function () { +(RavenTestContext.is60Server ? describe.skip : describe)("ClientGraphQueriesTest", function () { let store: IDocumentStore; @@ -233,4 +233,4 @@ class FriendDescriptor { this.friendsSince = friendsSince; this.friendId = friendId; } -} \ No newline at end of file +} diff --git a/test/Ported/Issues/RavenDB_8328Test.ts b/test/Ported/Issues/RavenDB_8328Test.ts index 526362a6..27f656ac 100644 --- a/test/Ported/Issues/RavenDB_8328Test.ts +++ b/test/Ported/Issues/RavenDB_8328Test.ts @@ -1,12 +1,12 @@ import * as assert from "assert"; -import { testContext, disposeTestDocumentStore } from "../../Utils/TestUtil"; +import { testContext, disposeTestDocumentStore, RavenTestContext } from "../../Utils/TestUtil"; import { IDocumentStore, PointField, WktField, QueryStatistics, } from "../../../src"; -describe("Issue RavenDB-8328", function () { +(RavenTestContext.is60Server ? describe.skip : describe)("Issue RavenDB-8328", function () { let store: IDocumentStore; diff --git a/test/Ported/RevisionsTest.ts b/test/Ported/RevisionsTest.ts index 7be3d763..41051613 100644 --- a/test/Ported/RevisionsTest.ts +++ b/test/Ported/RevisionsTest.ts @@ -1,5 +1,5 @@ import * as assert from "assert"; -import { testContext, disposeTestDocumentStore } from "../Utils/TestUtil"; +import { testContext, disposeTestDocumentStore, RavenTestContext } from "../Utils/TestUtil"; import { ConfigureRevisionsOperation, @@ -21,7 +21,7 @@ import { RevisionsResult } from "../../src/Documents/Operations/Revisions/Revisi import { delay } from "../../src/Utility/PromiseUtil"; -describe("RevisionsTest", function () { +(RavenTestContext.is60Server ? describe.skip : describe)("RevisionsTest", function () { let store: IDocumentStore; diff --git a/test/Ported/Server/Replication/PullReplicationTest.ts b/test/Ported/Server/Replication/PullReplicationTest.ts index f1ceb07f..88753bab 100644 --- a/test/Ported/Server/Replication/PullReplicationTest.ts +++ b/test/Ported/Server/Replication/PullReplicationTest.ts @@ -238,7 +238,7 @@ import { delay } from "../../../../src/Utility/PromiseUtil"; const pull: PullReplicationAsSink = { database: hub2.database, connectionStringName: "ConnectionString2-" + sink.database, - hubDefinitionName: definitionName2, + hubName: definitionName2, taskId: pullTasks[0].taskId, mode: "HubToSink" }; @@ -366,7 +366,7 @@ import { delay } from "../../../../src/Utility/PromiseUtil"; const pull: PullReplicationAsSink = { database: hub.database, connectionStringName: "ConnectionString-" + sink.database, - hubDefinitionName: definitionName, + hubName: definitionName, disabled: true, taskId: pullTasks[0].taskId, mode: "HubToSink" @@ -514,7 +514,7 @@ async function setupPullReplication(remoteName: string, sink: IDocumentStore, .. const pull: PullReplicationAsSink = { database: store.database, connectionStringName: "ConnectionString-" + store.database, - hubDefinitionName: remoteName, + hubName: remoteName, mode: "HubToSink" }; diff --git a/test/Ported/Suggestions/SuggestionsTest.ts b/test/Ported/Suggestions/SuggestionsTest.ts index d74e1879..aa40ca29 100644 --- a/test/Ported/Suggestions/SuggestionsTest.ts +++ b/test/Ported/Suggestions/SuggestionsTest.ts @@ -1,10 +1,10 @@ import * as assert from "assert"; import { IDocumentStore, IndexDefinition, IndexFieldOptions, PutIndexesOperation, SuggestionOptions } from "../../../src"; -import { disposeTestDocumentStore, testContext } from "../../Utils/TestUtil"; +import { disposeTestDocumentStore, RavenTestContext, testContext } from "../../Utils/TestUtil"; import { Users_ByName } from "../Indexing/IndexesFromClientTest"; -describe("SuggestionsTest", function () { +(RavenTestContext.is60Server ? describe.skip : describe)("SuggestionsTest", function () { let store: IDocumentStore; diff --git a/test/Utils/ReplicationTestContext.ts b/test/Utils/ReplicationTestContext.ts index 89aab598..edb21674 100644 --- a/test/Utils/ReplicationTestContext.ts +++ b/test/Utils/ReplicationTestContext.ts @@ -62,7 +62,7 @@ export class ReplicationTestContext { let op: IMaintenanceOperation; - if ("hubDefinitionName" in watcher) { + if ("hubName" in watcher) { op = new UpdatePullReplicationAsSinkOperation(watcher as PullReplicationAsSink); } else { op = new UpdateExternalReplicationOperation(watcher); diff --git a/test/Utils/TestUtil.ts b/test/Utils/TestUtil.ts index a1d8a306..b7e3a7e0 100644 --- a/test/Utils/TestUtil.ts +++ b/test/Utils/TestUtil.ts @@ -49,7 +49,7 @@ function logOnUncaughtAndUnhandled() { class TestServiceLocator extends RavenServerLocator { public getCommandArguments() { const cliOpts = [ - "--ServerUrl=http://127.0.0.1:0", + "--ServerUrl=http://127.0.0.1:0", "--ServerUrl.Tcp=tcp://127.0.0.1:38884", "--Features.Availability=Experimental" ]; @@ -137,6 +137,8 @@ export class RavenTestContext extends RavenTestDriver implements IDisposable { public static isPullRequest = !process.env["RAVEN_License"]; + public static is60Server = process.env["RAVENDB_SERVER_VERSION"] >= "6.0"; + private readonly _locator: RavenServerLocator; private readonly _securedLocator: RavenServerLocator;