Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Provided ability to set either of raw body of several dody data to `E… #315

Merged
merged 4 commits into from
Jun 12, 2024

Conversation

Nikita-Smirnov-Exactpro
Copy link
Member

…vent` builder

Copy link

github-actions bot commented Jun 12, 2024

Test Results

 31 files  ±0   31 suites  ±0   4s ⏱️ -1s
396 tests +5  396 ✅ +5  0 💤 ±0  0 ❌ ±0 
450 runs  +5  450 ✅ +5  0 💤 ±0  0 ❌ ±0 

Results for commit 63122d8. ± Comparison against base commit 236125c.

This pull request removes 88 and adds 50 tests. Note that renamed tests count towards both.
        nanos: 132287000
        nanos: 132585000
        nanos: 132639000
        nanos: 132688000
        nanos: 160009000
        nanos: 160045000
        nanos: 173345000
        nanos: 173432000
        nanos: 179294000
        nanos: 179376000
…
com.exactpro.th2.common.event.TestEvent ‑ add body data when raw body is already set()
com.exactpro.th2.common.event.TestEvent ‑ set description when raw body is already set()
com.exactpro.th2.common.event.TestEvent ‑ set raw body again when raw body is already set()
com.exactpro.th2.common.event.TestEvent ‑ set raw body when body data is already added()
com.exactpro.th2.common.event.TestEvent ‑ set raw body when description is already set()
com.exactpro.th2.common.schema.filter.strategy.impl.TestAnyMessageFilterStrategy ‑ [1] message {
  metadata {
    id {
      connection_id {
        session_alias: "test-alias"
      }
      book_name: "test_book"
      timestamp {
        seconds: 1718201867
        nanos: 517697000
      }
    }
    message_type: "test"
    properties {
      key: "prop-field1"
      value: "prop-value1"
    }
    properties {
      key: "prop-field2"
      value: "prop-value2"
    }
  }
}
, true
com.exactpro.th2.common.schema.filter.strategy.impl.TestAnyMessageFilterStrategy ‑ [1] message {
  metadata {
    id {
      connection_id {
        session_alias: "test-alias"
      }
      book_name: "test_book"
      timestamp {
        seconds: 1718201867
        nanos: 558191000
      }
    }
    message_type: "test"
    protocol: "HTTP"
  }
}
, true
com.exactpro.th2.common.schema.filter.strategy.impl.TestAnyMessageFilterStrategy ‑ [1] message {
  metadata {
    id {
      connection_id {
        session_alias: "test-alias"
      }
      book_name: "test_book"
      timestamp {
        seconds: 1718201867
        nanos: 562661000
      }
    }
    message_type: "test"
  }
  fields {
    key: "test-field1"
    value {
      simple_value: "test-value1"
    }
  }
  fields {
    key: "test-field2"
    value {
      simple_value: "test-value2"
    }
  }
}
, true
com.exactpro.th2.common.schema.filter.strategy.impl.TestAnyMessageFilterStrategy ‑ [1] message {
  metadata {
    id {
      connection_id {
        session_alias: "test-alias"
      }
      book_name: "test_book"
      timestamp {
        seconds: 1718201867
        nanos: 571086000
      }
    }
    message_type: "test"
  }
}
, true
com.exactpro.th2.common.schema.filter.strategy.impl.TestAnyMessageFilterStrategy ‑ [1] message {
  metadata {
    id {
      connection_id {
        session_alias: "test-alias"
      }
      book_name: "test_book"
      timestamp {
        seconds: 1718201867
        nanos: 579867000
      }
    }
    message_type: "test"
    properties {
      key: "test-property"
      value: "property-value"
    }
  }
}
, true
…

♻️ This comment has been updated with latest results.

@Nikita-Smirnov-Exactpro Nikita-Smirnov-Exactpro marked this pull request as ready for review June 12, 2024 08:48
OptimumCode
OptimumCode previously approved these changes Jun 12, 2024
src/main/java/com/exactpro/th2/common/event/Event.java Outdated Show resolved Hide resolved
src/main/java/com/exactpro/th2/common/event/Event.java Outdated Show resolved Hide resolved
Copy link
Member

@OptimumCode OptimumCode left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please, update tests

@Nikita-Smirnov-Exactpro Nikita-Smirnov-Exactpro merged commit e7029ac into master Jun 12, 2024
11 of 12 checks passed
@Nikita-Smirnov-Exactpro Nikita-Smirnov-Exactpro deleted the event-raw-body branch June 12, 2024 14:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants