Comcast / plax

A test automation engine for messaging systems
Apache License 2.0
7 stars 14 forks source link

Issue #134: Octane report plugin support #133

Closed satwic007 closed 3 years ago

satwic007 commented 3 years ago

Description: This PR adds Octane plugin code. We just need to add the config to the plaxrun file and plaxrun takes care of the rest.

Sample usage:

  1. I added config (referred in the doc/octane_plugin.md below) to cmd/plaxrun/demos/waitrun.yaml and ran the command - plaxrun -run cmd/plaxrun/demos/waitrun.yaml -dir demos -g wait-no-prompt -json
  2. Octane plugin - console log:
    <test_result>
    <product_areas>
    <product_area_ref id="123456"></product_area_ref>
    </product_areas>
    <test_runs>
    <test_run name="test-wait waitrun-0.0.1:wait-no-prompt:wait" duration="606" status="Failed" started="1632173133000">
    <test_fields>
    <test_field type="Framework" value="plax"></test_field>
    </test_fields>
    <error>Broken: Err: Broken: phase phase1: Broken: step 5: Broken: Javascript problem: failure: too slow: 605</error>
    </test_run>
    </test_runs>
    </test_result>
  3. Equivalent Plax console log:
    {
    "name": "waitrun",
    "version": "0.0.1",
    "testsuite": [
    {
      "name": "waitrun-0.0.1:wait-no-prompt:wait",
      "tests": 1,
      "passed": 0,
      "skipped": 0,
      "failures": 0,
      "errors": 1,
      "testcase": [
        {
          "name": "test-wait",
          "file": "/Users/smudig200/Documents/comcastgit/octane_plugin/plax/demos/test-wait.yaml",
          "status": "error",
          "time": 606178000,
          "started": "2021-09-20T21:25:33.730331Z",
          "message": "Broken: Err: Broken: phase phase1: Broken: step 5: Broken: Javascript problem: failure: too slow: 605"
        }
      ],
      "timestamp": "2021-09-20T21:25:33.728889Z",
      "time": 607622000
    }
    ],
    "tests": 1,
    "passed": 0,
    "skipped": 0,
    "failures": 0,
    "errors": 1,
    "timestamp": "2021-09-20T21:25:33.728736Z",
    "time": 607783000
    }
CLAassistant commented 3 years ago

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

:white_check_mark: satwic007
:x: smudig200


smudig200 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

satwic007 commented 3 years ago

Created a new pull request with clean editor info and updated it with suggested changes.

robertfarnum commented 3 years ago

Closing due to duplicate PR&137