Revert "[s3] add support for testing on macOS (tensorflow#1253)"

yongtang · yongtang · commit 4bd369f99bb5 · 2021-01-18T15:34:39.000-08:00
This reverts commit 81789bd. Signed-off-by: Yong Tang <yong.tang.github@outlook.com>
diff --git a/tests/test_aws/aws_test.sh b/tests/test_aws/aws_test.sh
@@ -17,11 +17,10 @@
 set -e
 set -o pipefail
 
-export LOCALSTACK_VERSION=0.12.2
-cd tests/test_aws
-TMPDIR=/private$TMPDIR docker-compose up -d
-cd -
-echo "Waiting for 20 secs until localstack is up and running"
-sleep 20
+LOCALSTACK_VERSION=0.12.2
+docker pull localstack/localstack:$LOCALSTACK_VERSION
+docker run -d --rm --net=host --name=tensorflow-io-aws localstack/localstack:$LOCALSTACK_VERSION
+echo "Waiting for 10 secs until localstack is up and running"
+sleep 10
 echo "Localstack up"
 exit 0
diff --git a/tests/test_aws/docker-compose.yml b/tests/test_aws/docker-compose.yml
diff --git a/tests/test_s3_eager.py b/tests/test_s3_eager.py
@@ -24,8 +24,8 @@
 
 
 @pytest.mark.skipif(
-    sys.platform in ("win32"),
-    reason="TODO Localstack not setup properly on Windows yet",
+    sys.platform in ("win32", "darwin"),
+    reason="TODO Localstack not setup properly on macOS/Windows yet",
 )
 def test_read_file():
     """Test case for reading S3"""

Original file line number	Diff line number	Diff line change
`@@ -24,8 +24,8 @@`
`24`	`24`
`25`	`25`
`26`	`26`	`@pytest.mark.skipif(`
`27`		`- sys.platform in ("win32"),`
`28`		`- reason="TODO Localstack not setup properly on Windows yet",`
	`27`	`+ sys.platform in ("win32", "darwin"),`
	`28`	`+ reason="TODO Localstack not setup properly on macOS/Windows yet",`
`29`	`29`	`)`
`30`	`30`	`def test_read_file():`
`31`	`31`	`"""Test case for reading S3"""`