Skip to content

Commit

Permalink
8249700: java/io/File/GetXSpace.java should be added to exclude list,…
Browse files Browse the repository at this point in the history
… and not @Ignore-d

Reviewed-by: alanb
  • Loading branch information
iignatev committed Jul 20, 2020
1 parent af0d6d2 commit 8d97637
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 8 deletions.
1 change: 1 addition & 0 deletions test/jdk/ProblemList.txt
Original file line number Original file line Diff line number Diff line change
Expand Up @@ -586,6 +586,7 @@ javax/management/remote/mandatory/connection/MultiThreadDeadLockTest.java 822733
# jdk_io # jdk_io


java/io/pathNames/GeneralWin32.java 8180264 windows-all java/io/pathNames/GeneralWin32.java 8180264 windows-all
java/io/File/GetXSpace.java 6501010,8249703 windows-all,macosx-all


############################################################################ ############################################################################


Expand Down
3 changes: 1 addition & 2 deletions test/jdk/java/io/File/GetXSpace.java
Original file line number Original file line Diff line number Diff line change
Expand Up @@ -24,7 +24,6 @@
/** /**
* @test * @test
* @bug 4057701 6286712 6364377 * @bug 4057701 6286712 6364377
* @ignore until 6492634 and 6501010 is fixed
* @run build GetXSpace * @run build GetXSpace
* @run shell GetXSpace.sh * @run shell GetXSpace.sh
* @summary Basic functionality of File.get-X-Space methods. * @summary Basic functionality of File.get-X-Space methods.
Expand Down Expand Up @@ -135,7 +134,7 @@ private static ArrayList space(String f) throws IOException {
ArrayList al = new ArrayList(); ArrayList al = new ArrayList();


Process p = null; Process p = null;
String cmd = "df -k" + (f == null ? "" : " " + f); String cmd = "df -k -P" + (f == null ? "" : " " + f);
p = Runtime.getRuntime().exec(cmd); p = Runtime.getRuntime().exec(cmd);
BufferedReader in = new BufferedReader BufferedReader in = new BufferedReader
(new InputStreamReader(p.getInputStream())); (new InputStreamReader(p.getInputStream()));
Expand Down
12 changes: 6 additions & 6 deletions test/jdk/java/io/File/GetXSpace.sh
Original file line number Original file line Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@
# set platform-dependent variable # set platform-dependent variable
OS=`uname -s` OS=`uname -s`
case "$OS" in case "$OS" in
Linux ) TMP=/tmp ;; Linux | Darwin ) TMP=/tmp ;;
Windows_98 ) return ;; Windows_98 ) return ;;
Windows* ) SID=`sid`; TMP="c:/temp" ;; Windows* ) SID=`sid`; TMP="c:/temp" ;;
* ) * )
Expand All @@ -49,11 +49,11 @@ allow() {
case "$OS" in case "$OS" in
Windows* ) chacl -g ${SID}:f $* ;; Windows* ) chacl -g ${SID}:f $* ;;
* ) chmod 777 $* ;; * ) chmod 777 $* ;;
esac esac
} }


runTest() { runTest() {
${TESTJAVA}/bin/java ${TESTVMOPTS} -cp ${TESTCLASSES} GetXSpace $* ${TESTJAVA}/bin/java ${TESTVMOPTS} -cp ${TESTCLASSES} GetXSpace $*
if [ $? -eq 0 ] if [ $? -eq 0 ]
then echo "Passed" then echo "Passed"
else else
Expand All @@ -64,17 +64,17 @@ runTest() {


# df output # df output
runTest runTest

# readable file in an unreadable directory # readable file in an unreadable directory
mkdir ${TMP1} mkdir -p ${TMP1}
touch ${TMP1}/foo touch ${TMP1}/foo
deny ${TMP1} deny ${TMP1}
runTest ${TMP1}/foo runTest ${TMP1}/foo
allow ${TMP1} allow ${TMP1}
rm -rf ${TMP1} rm -rf ${TMP1}


if [ ${FAIL} -ne 0 ] if [ ${FAIL} -ne 0 ]
then then
echo "" echo ""
echo "${FAIL} test(s) failed" echo "${FAIL} test(s) failed"
exit 1 exit 1
Expand Down

0 comments on commit 8d97637

Please sign in to comment.