Look at adding helpers for integration tests

Open
#52 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
5/5
Estimated time
Over a week
Newbie friendliness
25/100
Issue type
Feature
Clarity
Needs clarification
Activity status
Stale
Tech stack
rails, ruby
Domain
testing

Research direction

Start by reading the linked ActionController::TestCase documentation and comparing it with ActionDispatch::IntegrationTest. Then inspect the gem's existing test helpers to determine what a second integration-test-oriented helper set would need to support; done means the required scope and behavior are defined and implemented with coverage for integration tests.

Written by the indexing model from the issue text.

Description

As of recent versions of Rails, they've begun discouraging the use of ActionController::TestCase for new tests and encouraging the use of integration tests based around ActionDispatch::IntegrationTest. The helpers in this gem are still based around the previous style; we may need to look at implementing a second set of helpers to enable the use of integration tests.

See: https://api.rubyonrails.org/classes/ActionController/TestCase.html

Dominant language
Ruby
Stars
68
Forks
9
No merged PRs in 30d

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from github/chatops-controller

All issues in github/chatops-controller

Similar issues

More Ruby issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.