Home Original page

test: require common module only once by Trott · Pull Request #4611 · nodejs/node

@Trott 

Two tests were requiring the common module twice. This removes the
duplicate require statement in the tests.

@Trott added the test

Issues and PRs related to the tests.

label

Jan 10, 2016

jasnell pushed a commit that referenced this pull request

Jan 11, 2016 
Two tests were requiring the common module twice. This removes the
duplicate require statement in the tests.

PR-URL: #4611
Reviewed-By: Brian White <mscdex@mscdex.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
Reviewed-By: James M Snell <jasnell@gmail.com>

MylesBorins pushed a commit that referenced this pull request

Jan 11, 2016 
Two tests were requiring the common module twice. This removes the
duplicate require statement in the tests.

PR-URL: #4611
Reviewed-By: Brian White <mscdex@mscdex.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
Reviewed-By: James M Snell <jasnell@gmail.com>

MylesBorins pushed a commit that referenced this pull request

Jan 12, 2016 
Two tests were requiring the common module twice. This removes the
duplicate require statement in the tests.

PR-URL: #4611
Reviewed-By: Brian White <mscdex@mscdex.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
Reviewed-By: James M Snell <jasnell@gmail.com>

MylesBorins pushed a commit that referenced this pull request

Jan 28, 2016 
Two tests were requiring the common module twice. This removes the
duplicate require statement in the tests.

PR-URL: #4611
Reviewed-By: Brian White <mscdex@mscdex.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
Reviewed-By: James M Snell <jasnell@gmail.com>

MylesBorins pushed a commit that referenced this pull request

Feb 11, 2016 
Two tests were requiring the common module twice. This removes the
duplicate require statement in the tests.

PR-URL: #4611
Reviewed-By: Brian White <mscdex@mscdex.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
Reviewed-By: James M Snell <jasnell@gmail.com>

MylesBorins pushed a commit to MylesBorins/node that referenced this pull request

Feb 11, 2016 
Two tests were requiring the common module twice. This removes the
duplicate require statement in the tests.

PR-URL: nodejs#4611
Reviewed-By: Brian White <mscdex@mscdex.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
Reviewed-By: James M Snell <jasnell@gmail.com>

MylesBorins pushed a commit to MylesBorins/node that referenced this pull request

Feb 15, 2016 
Two tests were requiring the common module twice. This removes the
duplicate require statement in the tests.

PR-URL: nodejs#4611
Reviewed-By: Brian White <mscdex@mscdex.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
Reviewed-By: James M Snell <jasnell@gmail.com>

scovetta pushed a commit to scovetta/node that referenced this pull request

Apr 2, 2016 
Two tests were requiring the common module twice. This removes the
duplicate require statement in the tests.

PR-URL: nodejs#4611
Reviewed-By: Brian White <mscdex@mscdex.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
Reviewed-By: James M Snell <jasnell@gmail.com>

@Trott Trott deleted the redecl-common branch

January 13, 2022 22:31