Skip to content

Add CSV response helper for controller specs

drew stachon requested to merge rspec-csv-response-helper into master

What does this MR do?

This MR adds a spec helper for us to use in a new controller spec. In the feature MR review, it was suggested that we refactor the logic for parsing a CSV response out into a helper the same way we do a JSON response. This is being sent independently because it's small and easily changed in advance.

Suggestion from original MR: !38520 (comment 392466758)

Screenshots

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • ~~Label as security and @ mention @gitlab-com/gl-security/appsec~~
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team

Merge request reports

Loading