Skip to content

Commit

Permalink
FIxed if condition
Browse files Browse the repository at this point in the history
  • Loading branch information
criminosis committed Jul 13, 2024
1 parent dada863 commit bc94862
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 9 deletions.
12 changes: 6 additions & 6 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -42,27 +42,27 @@ jobs:
command: fmt
args: --all -- --check
- name: Run cargo test with tokio
if: ${{ matrix.gremlin-server }} == 3.5.7
if: ${{ matrix.gremlin-server }} == "3.5.7"
uses: actions-rs/cargo@v1
with:
command: test
args: --manifest-path gremlin-client/Cargo.toml --features=tokio-runtime
- name: Run cargo test with async-std
if: ${{ matrix.gremlin-server }} == 3.5.7
if: ${{ matrix.gremlin-server }} == "3.5.7"
uses: actions-rs/cargo@v1
with:
command: test
args: --manifest-path gremlin-client/Cargo.toml --features=async-std-runtime
# MergeV as a step doesn't exist in 3.5.x, so selectively run those tests
- name: Run cargo test with tokio
if: ${{ matrix.gremlin-server }} == 3.6.5
if: ${{ matrix.gremlin-server }} != "3.5.7"
uses: actions-rs/cargo@v1
with:
command: test
args: --manifest-path gremlin-client/Cargo.toml --features=tokio-runtime,merge_v_tests
args: --manifest-path gremlin-client/Cargo.toml --features=tokio-runtime,merge_tests
- name: Run cargo test with async-std
if: ${{ matrix.gremlin-server }} == 3.6.5
if: ${{ matrix.gremlin-server }} != "3.5.7"
uses: actions-rs/cargo@v1
with:
command: test
args: --manifest-path gremlin-client/Cargo.toml --features=async-std-runtime,merge_v_tests
args: --manifest-path gremlin-client/Cargo.toml --features=async-std-runtime,merge_tests
10 changes: 7 additions & 3 deletions gremlin-client/tests/integration_traversal.rs
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,10 @@ use std::convert::TryInto;

use gremlin_client::process::traversal::{traversal, Order, __};
use gremlin_client::structure::{Cardinality, List, Map, Pop, TextP, Vertex, VertexProperty, P, T};

#[cfg(feature = "merge_tests")]
use gremlin_client::structure::Merge;

use gremlin_client::utils;

mod common;
Expand All @@ -20,7 +24,7 @@ fn test_simple_vertex_traversal() {
}

#[test]
#[cfg(feature = "merge_v_tests")]
#[cfg(feature = "merge_tests")]
fn test_merge_v_no_options() {
let g = traversal().with_remote(graph());
let expected_id = 1_000i64;
Expand Down Expand Up @@ -65,7 +69,7 @@ fn test_merge_v_no_options() {
}

#[test]
#[cfg(feature = "merge_v_tests")]
#[cfg(feature = "merge_tests")]
fn test_merge_v_options() {
let g = traversal().with_remote(graph());
let expected_label = "test_merge_v_options_label";
Expand Down Expand Up @@ -131,7 +135,7 @@ fn test_merge_v_options() {
}

#[test]
#[cfg(feature = "merge_v_tests")]
#[cfg(feature = "merge_tests")]
fn test_merge_v_start_step() {
let g = traversal().with_remote(graph());
let expected_id = 10000i64;
Expand Down

0 comments on commit bc94862

Please sign in to comment.