tendermint-rpc: use top level xdescribe() call

Author: dynst

packages/tendermint-rpc/src/comet38/comet38client.spec.ts

@@ -851,10 +851,10 @@ function websocketTestSuite(rpcFactory: () => RpcClient, expected: ExpectedValue
   });
 }
 
-describe("Comet38Client with CometBFT 0.38 backend", () => {
+(tendermintEnabled ? describe : xdescribe)("Comet38Client with CometBFT 0.38 backend", () => {
   const { url, expected } = tendermintInstances[38];
 
-  (tendermintEnabled ? it : xit)("can connect to a given url", async () => {
+  it("can connect to a given url", async () => {
     // http connection
     {
       const client = await Comet38Client.connect("http://" + url);
@@ -872,11 +872,11 @@ describe("Comet38Client with CometBFT 0.38 backend", () => {
     }
   });
 
-  (tendermintEnabled ? describe : xdescribe)("With HttpClient", () => {
+  describe("With HttpClient", () => {
     defaultTestSuite(() => new HttpClient("http://" + url), expected);
   });
 
-  (tendermintEnabled ? describe : xdescribe)("With WebsocketClient", () => {
+  describe("With WebsocketClient", () => {
     // don't print out WebSocket errors if marked pending
     const onError = globalThis.process?.env.TENDERMINT_ENABLED ? console.error : () => 0;
     const factory = (): WebsocketClient => new WebsocketClient("ws://" + url, onError);

packages/tendermint-rpc/src/tendermint34/tendermint34client.spec.ts

@@ -846,10 +846,10 @@ function websocketTestSuite(rpcFactory: () => RpcClient, expected: ExpectedValue
   });
 }
 
-describe("Tendermint34Client", () => {
+(tendermintEnabled ? describe : xdescribe)("Tendermint34Client", () => {
   const { url, expected } = tendermintInstances[34];
 
-  (tendermintEnabled ? it : xit)("can connect to a given url", async () => {
+  it("can connect to a given url", async () => {
     // http connection
     {
       const client = await Tendermint34Client.connect("http://" + url);
@@ -867,11 +867,11 @@ describe("Tendermint34Client", () => {
     }
   });
 
-  (tendermintEnabled ? describe : xdescribe)("With HttpClient", () => {
+  describe("With HttpClient", () => {
     defaultTestSuite(() => new HttpClient("http://" + url), expected);
   });
 
-  (tendermintEnabled ? describe : xdescribe)("With WebsocketClient", () => {
+  describe("With WebsocketClient", () => {
     // don't print out WebSocket errors if marked pending
     const onError = globalThis.process?.env.TENDERMINT_ENABLED ? console.error : () => 0;
     const factory = (): WebsocketClient => new WebsocketClient("ws://" + url, onError);

packages/tendermint-rpc/src/tendermint37/tendermint37client.spec.ts

@@ -848,10 +848,10 @@ function websocketTestSuite(rpcFactory: () => RpcClient, expected: ExpectedValue
   });
 }
 
-describe("Tendermint37Client", () => {
+(tendermintEnabled ? describe : xdescribe)("Tendermint37Client", () => {
   const { url, expected } = tendermintInstances[37];
 
-  (tendermintEnabled ? it : xit)("can connect to a given url", async () => {
+  it("can connect to a given url", async () => {
     // http connection
     {
       const client = await Tendermint37Client.connect("http://" + url);
@@ -869,11 +869,11 @@ describe("Tendermint37Client", () => {
     }
   });
 
-  (tendermintEnabled ? describe : xdescribe)("With HttpClient", () => {
+  describe("With HttpClient", () => {
     defaultTestSuite(() => new HttpClient("http://" + url), expected);
   });
 
-  (tendermintEnabled ? describe : xdescribe)("With WebsocketClient", () => {
+  describe("With WebsocketClient", () => {
     // don't print out WebSocket errors if marked pending
     const onError = globalThis.process?.env.TENDERMINT_ENABLED ? console.error : () => 0;
     const factory = (): WebsocketClient => new WebsocketClient("ws://" + url, onError);