Problem/Motivation

No test coverage found.

Steps to reproduce

Install module and check for test coverage. No test coverage found.

CommentFileSizeAuthor
#4 DeferJsFormTest.txt6.93 KBshubh02
#2 DeferJsFormTest.test6.93 KBshubh02

Issue fork deferjs-3538496

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

shubh02 created an issue. See original summary.

shubh02’s picture

shubh02
Primary language English
commented
Status: Active » Needs review
StatusFileSize
new DeferJsFormTest.test6.93 KB

Uploaded wrong file

anurag_nagar made their first commit to this issue’s fork.

shubh02’s picture

shubh02
Primary language English
commented
StatusFileSize
new DeferJsFormTest.txt6.93 KB

Proposed resolution

Adding test coverage in existing module.
Step 1 - Create a tests folder on root of module.
Step 2 - Create src folder in tests folder.
Step 3 - Create Unit folder in src folder under tests folder.
Step 4 - Create Form folder in Unit folder which is under src/tests.

So the final path will be /tests/src/Unit/Form
Step 5 - Download DeferJsFormTest.txt file and rename it with DeferJsFormTest.php
Step 6 - Place this test file DeferJsFormTest.php on path /tests/src/Unit/Form

Now test coverage is available for this module.

anurag_nagar opened merge request !2

anurag_nagar’s picture

anurag_nagar commented
Status: Needs review » Reviewed & tested by the community

@shubh02 I have checked the test file which seems to be covering test case for number of lines in DeferJsForm , I have raise a MR against it.

vinodhini.e’s picture

vinodhini.e commented

Hi, I have installed the module on a Drupal 11.2.2 and applied the merge request #2

Reviewed the test file, which covers test cases for the number of lines in the DeferJsForm.

Thanks.

anurag_nagar changed the visibility of the branch 3538496-adding-test-coverage to hidden.

jeetmail72 closed merge request !2

jeetmail72 reopened merge request !2

jeetmail72 closed merge request !2

jeetmail72’s picture

jeetmail72
He/Him
Primary language English
Location Noida
commented
Assigned: shubh02 » Unassigned
Status: Reviewed & tested by the community » Fixed

Now that this issue is closed, please review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, please credit people who helped resolve this issue.

jeetmail72’s picture

jeetmail72
He/Him
Primary language English
Location Noida
commented
Status: Fixed » Closed (fixed)