41 tests are currently skipped when run with AIRPLANE=true. Most if not all of these tests depend on external websites, which means if they change their behavior or are temporarily down, our tests will fail.

Instead of depending on outside websites I suggest we spawn our own servers as we need them, it is simple enough.

See #1083 for an example pull request / change set.

I think these are all occurrences where we use the {skip: process.env.AIRPLANE} setting.

If you’d like to help out, great! Please do the following

1. comment below which one of these (can be multiple) you want to work on
2. create an issue with the list of occurrences you want to work on and reference this issue (Remove AIRPLANE mode for testing - make all tests run offline #1077)
3. Then update your comment below with a link to that issue, I’ll keep the "Available" list updated

Claimed / in progress

🤷‍♀️

Available

🤷‍♂️

Done

• tests/test_https_allowunmocked.js#L9 @jlilja Nock-1077 Remove airplane mode #1209
• tests/test_https_allowunmocked.js#L27 @jlilja Nock-1077 Remove airplane mode #1209
• tests/test_recorder.js#L93 @paulmelnikow PR: test(test_recorder): De-airplane remaining recorder tests #1283
• tests/test_recorder.js#L140 @paulmelnikow PR: test(test_recorder): De-airplane remaining recorder tests #1283
• tests/test_recorder.js#L195 @paulmelnikow PR: test(test_recorder): De-airplane remaining recorder tests #1283
• tests/test_recorder.js#L209 @paulmelnikow PR: test(test_recorder): De-airplane remaining recorder tests #1283
• tests/test_recorder.js#L236 @paulmelnikow PR: test(test_recorder): De-airplane remaining recorder tests #1283
• tests/test_recorder.js#L340 @paulmelnikow PR: test(test_recorder): De-airplane remaining recorder tests #1283
• tests/test_recorder.js#L374 @paulmelnikow PR: test(test_recorder): De-airplane remaining recorder tests #1283
• tests/test_recorder.js#L409 @paulmelnikow PR: test(test_recorder): De-airplane remaining recorder tests #1283
• tests/test_recorder.js#L457 @paulmelnikow PR: test(test_recorder): De-airplane remaining recorder tests #1283
• tests/test_recorder.js#L502 test(test_recorder): De-airplane more recorder tests #1281
• tests/test_recorder.js#L555 test(test_recorder): De-airplane more recorder tests #1281
• tests/test_recorder.js#L586 test(test_recorder): De-airplane more recorder tests #1281
• tests/test_recorder.js#L621 test(test_recorder): De-airplane more recorder tests #1281
• tests/test_recorder.js#L656 test(test_recorder): De-airplane more recorder tests #1281
• tests/test_recorder.js#L689 test(test_recorder): De-airplane more recorder tests #1281
• tests/test_recorder.js#L710 test(test_recorder): De-airplane more recorder tests #1281
• tests/test_recorder.js#L62 remove airplane mode for "records objects" test #1279
• tests/test_recorder.js#L852 test(test_recorder.js): Remove AIRPLANE from a bunch of tests #1280
• tests/test_recorder.js#L733 test(test_recorder.js): Remove AIRPLANE from a bunch of tests #1280
• tests/test_recorder.js#L796 test(test_recorder.js): Remove AIRPLANE from a bunch of tests #1280
• tests/test_recorder.js#L824 test(test_recorder.js): Remove AIRPLANE from a bunch of tests #1280
• tests/test_nock_off.js#L6 remove AIRPLANE mode from test_nock_off #1277 @gr2m
• tests/test_recorder.js#L36 test(test_recorder.js): Fix records test #1278 @RichardLitt
• tests/test_intercept.js#L2885
• tests/test_intercept.js#L2902
• tests/test_intercept.js#L2684
• tests/test_intercept.js#L2728
• tests/test_intercept.js#L2745
• tests/test_intercept.js#L3673
• tests/test_back.js#L154 – PR: remove airplane dependency for 'tests/test_back#L154' #1082
• Add "Do not copy" comment to all tests that still use "{skip: process.env.AIRPLANE}" setting
• tests/test_back.js#L137 – PR: test: remove airplane dependency for "tests/test_back#L137" #1083
• tests/test_back.js#L188 - PR: Remove airplane dependency from "tests/test_back.js#L188" #1087
• tests/test_back.js#L235 - PR: Remove airplane dependency from "tests/test_back.js#L235" #1088
• tests/test_back.js#L261 - PR: Remove airplane dependency from "tests/test_back.js#L261" #1090
• tests/test_back.js#L313 – PR: test: remove airplace dependency from tests/test_back.js#313 #1093
• tests/test_back_2.js#L31 - PR: Remove airplane dependency from "test_back_2.js" #1097
• tests/test_back_2.js#L53 - PR: Remove airplane dependency from "test_back_2.js" #1097
• tests/test_intercept.js#L2108 - PR: [WIP] Remove airplane dependency from "test_intercept.js" #1104
• tests/test_intercept.js#L2658 - PR: [WIP] Remove airplane dependency from "test_intercept.js" #1104