From 5e01d148f545df6ef874b30673dd49c4f78e4c69 Mon Sep 17 00:00:00 2001 From: Rich Trott Date: Wed, 26 Jul 2017 12:28:26 -0700 Subject: [PATCH] test: remove disabled tests directory There is only one test left in the disabled test directory and it does not seem worth saving. --- .eslintignore | 1 - test/README.md | 1 - test/disabled/test-https-loop-to-google.js | 48 ---------------------- 3 files changed, 50 deletions(-) delete mode 100644 test/disabled/test-https-loop-to-google.js diff --git a/.eslintignore b/.eslintignore index 26f48161f71f8c..a681716c667576 100644 --- a/.eslintignore +++ b/.eslintignore @@ -2,7 +2,6 @@ lib/internal/v8_prof_polyfill.js lib/punycode.js test/addons/??_*/ test/fixtures -test/disabled test/tmp*/ tools/eslint tools/icu diff --git a/test/README.md b/test/README.md index b5853bc7418070..8a74325e087c24 100644 --- a/test/README.md +++ b/test/README.md @@ -17,7 +17,6 @@ On how to run tests in this directory, see |cctest |Yes |C++ test that is run as part of the build process.| |common | |Common modules shared among many tests. [Documentation](./common/README.md)| |debugger |No |Tests for [debugger](https://nodejs.org/api/debugger.html)functionality along with some tests that require an addon to function properly.| -|disabled |No |Tests that have been disabled from running for various reasons.| |fixtures | |Test fixtures used in various tests throughout the test suite.| |gc |No |Tests for garbage collection related functionality.| |inspector |Yes |Tests for the V8 inspector integration.| diff --git a/test/disabled/test-https-loop-to-google.js b/test/disabled/test-https-loop-to-google.js deleted file mode 100644 index 1fd1048df1bdfe..00000000000000 --- a/test/disabled/test-https-loop-to-google.js +++ /dev/null @@ -1,48 +0,0 @@ -// Copyright Joyent, Inc. and other Node contributors. -// -// Permission is hereby granted, free of charge, to any person obtaining a -// copy of this software and associated documentation files (the -// "Software"), to deal in the Software without restriction, including -// without limitation the rights to use, copy, modify, merge, publish, -// distribute, sublicense, and/or sell copies of the Software, and to permit -// persons to whom the Software is furnished to do so, subject to the -// following conditions: -// -// The above copyright notice and this permission notice shall be included -// in all copies or substantial portions of the Software. -// -// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS -// OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF -// MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN -// NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, -// DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR -// OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE -// USE OR OTHER DEALINGS IN THE SOFTWARE. - -'use strict'; -// Failing test for https - -// Will fail with "socket hang up" for 4 out of 10 requests -// Tested on node 0.5.0-pre commit 9851574 - - -const common = require('../common'); -const https = require('https'); - -for (var i = 0; i < 10; ++i) { - https.get({ - host: 'www.google.com', - path: '/accounts/o8/id', - port: 443 - }, function(res) { - var data = ''; - res.on('data', function(chunk) { - data += chunk; - }); - res.on('end', function() { - console.log(res.statusCode); - }); - }).on('error', function(error) { - console.log(error); - }); -}