Skip to content
Snippets Groups Projects

Add integration test Spec

Merged Craig Smith requested to merge craigmsmith-add-integration-spec into master
All threads resolved!

What does this MR do?

To bring this project inline with the other analyzers, this MR adds an RSpec integration test to ensure the scanner is working as expected.

What are the relevant issue numbers?

This MR relates to but is not required by gitlab-org/gitlab#373921 (closed)

Does this MR meet the acceptance criteria?

Edited by Craig Smith

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Craig Smith requested review from @ahmed.hemdan

    requested review from @ahmed.hemdan

  • Craig Smith changed title from Add nodejs scan integration test to Add integration test Spec

    changed title from Add nodejs scan integration test to Add integration test Spec

  • Ahmed Hemdan
  • Craig Smith added 1 commit

    added 1 commit

    • cbab4a89 - Add integration spec instructions to README

    Compare with previous version

  • Craig Smith added 1 commit

    added 1 commit

    • 6d84b386 - Bring README inline with others

    Compare with previous version

  • Craig Smith added 2 commits

    added 2 commits

    • 2d1d2666 - Add nodejs scan integration test
    • f6d0ee42 - Add integration spec instructions to README

    Compare with previous version

  • Ahmed Hemdan approved this merge request

    approved this merge request

  • requested review from @jamesliu-gitlab

  • Ahmed Hemdan removed review request for @ahmed.hemdan

    removed review request for @ahmed.hemdan

  • James Liu approved this merge request

    approved this merge request

  • James Liu resolved all threads

    resolved all threads

  • James Liu mentioned in commit 1c802609

    mentioned in commit 1c802609

  • merged

  • Please register or sign in to reply
    Loading