[PATCH] D80838: [llvm-ar] Add more tests for errors in opening archives

Sameer Arora via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 3 07:06:14 PDT 2020


sameerarora101 updated this revision to Diff 268181.
sameerarora101 added a comment.

Adding `# UNSUPPORTED: system-windows` to the test case.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D80838/new/

https://reviews.llvm.org/D80838

Files:
  llvm/test/tools/llvm-ar/error-opening-archive.test
  llvm/test/tools/llvm-ar/print.test


Index: llvm/test/tools/llvm-ar/print.test
===================================================================
--- llvm/test/tools/llvm-ar/print.test
+++ llvm/test/tools/llvm-ar/print.test
@@ -68,7 +68,7 @@
 ## No archive:
 # RUN: not llvm-ar p 2>&1 \
 # RUN:   | FileCheck %s --check-prefix=NO-ARCHIVE
-# 
+#
 # NO-ARCHIVE: error: an archive name must be specified
 
 ## Archive does not exist:
Index: llvm/test/tools/llvm-ar/error-opening-archive.test
===================================================================
--- /dev/null
+++ llvm/test/tools/llvm-ar/error-opening-archive.test
@@ -0,0 +1,19 @@
+# UNSUPPORTED: system-windows
+
+# RUN: rm -rf %t && mkdir -p %t
+# RUN: echo file1 > %t/1.txt
+
+## No Permission:
+# RUN: llvm-ar rc %t/permission.b %t/1.txt
+# RUN: chmod 100 %t/permission.b
+# RUN: not llvm-ar p %t/permission.b 2>&1 | \
+# RUN:   FileCheck %s --check-prefix=NO-PERMISSION -DARCHIVE=%t/permission.b
+
+# NO-PERMISSION: error: error opening '[[ARCHIVE]]': {{[pP]}}ermission denied
+
+## Passing in a directory:
+# RUN: mkdir -p %t/tmpDir
+# RUN: not llvm-ar p %t/tmpDir 2>&1 | \
+# RUN:    FileCheck %s --check-prefix=IS-DIR -DARCHIVE=%t/tmpDir
+
+# IS-DIR: error: error opening '[[ARCHIVE]]': {{[iI]}}s a directory


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D80838.268181.patch
Type: text/x-patch
Size: 1243 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200603/991bbca1/attachment.bin>


More information about the llvm-commits mailing list