X-Git-Url: http://repo.macrolet.net/gitweb/?a=blobdiff_plain;f=tests%2Fstream.impure.lisp;h=78e331b021efd1d3d8da09d745f903ce3ed17a57;hb=e7dc41bb0e683c1ea329ce720cddd148da6e92e1;hp=7348bfea9f29030a02b75211b8a3519f722fe2c5;hpb=cbe488f1e264bc8f7b0501430b260db1887b055d;p=sbcl.git diff --git a/tests/stream.impure.lisp b/tests/stream.impure.lisp index 7348bfe..78e331b 100644 --- a/tests/stream.impure.lisp +++ b/tests/stream.impure.lisp @@ -6,7 +6,7 @@ ;;;; While most of SBCL is derived from the CMU CL system, the test ;;;; files (like this one) were written from scratch after the fork ;;;; from CMU CL. -;;;; +;;;; ;;;; This software is in the public domain and is provided with ;;;; absolutely no warranty. See the COPYING and CREDITS files for ;;;; more information. @@ -19,11 +19,11 @@ (locally (declare (optimize (safety 3))) (assert (raises-error? (make-two-way-stream (make-string-output-stream) - (make-string-output-stream)) - type-error)) + (make-string-output-stream)) + type-error)) (assert (raises-error? (make-two-way-stream (make-string-input-stream "foo") - (make-string-input-stream "bar")) - type-error)) + (make-string-input-stream "bar")) + type-error)) ;; the following two aren't actually guaranteed, because ANSI, as it ;; happens, doesn't say "should signal an error" for ;; MAKE-ECHO-STREAM. It's still good to have, but if future @@ -31,15 +31,15 @@ ;; MAKE-ECHO-STREAM clauses, consider simply removing these clauses ;; from the test. -- CSR, 2002-10-06 (assert (raises-error? (make-echo-stream (make-string-output-stream) - (make-string-output-stream)) - type-error)) + (make-string-output-stream)) + type-error)) (assert (raises-error? (make-echo-stream (make-string-input-stream "foo") - (make-string-input-stream "bar")) - type-error)) + (make-string-input-stream "bar")) + type-error)) (assert (raises-error? (make-concatenated-stream - (make-string-output-stream) - (make-string-input-stream "foo")) - type-error))) + (make-string-output-stream) + (make-string-input-stream "foo")) + type-error))) ;;; bug 225: STRING-STREAM was not a class (eval `(defgeneric bug225 (s) @@ -58,9 +58,9 @@ ;;; improper buffering on (SIGNED-BYTE 8) streams (fixed by David Lichteblau): (let ((p "signed-byte-8-test.data")) (with-open-file (s p - :direction :output - :element-type '(unsigned-byte 8) - :if-exists :supersede) + :direction :output + :element-type '(unsigned-byte 8) + :if-exists :supersede) (write-byte 255 s)) (with-open-file (s p :element-type '(signed-byte 8)) (assert (= (read-byte s) -1))) @@ -72,15 +72,15 @@ (stream (open p :direction :output :if-exists :error))) (assert (null (with-open-file (s p :direction :output :if-exists nil) s))) (assert (raises-error? - (with-open-file (s p :direction :output :if-exists :error)))) + (with-open-file (s p :direction :output :if-exists :error)))) (close stream) (delete-file p)) (assert (raises-error? (read-byte (make-string-input-stream "abc")) - type-error)) + type-error)) (assert (raises-error? (with-open-file (s "/dev/zero") - (read-byte s)) - type-error)) + (read-byte s)) + type-error)) ;;; bidirectional streams getting confused about their position (let ((p "bidirectional-stream-test")) (with-open-file (s p :direction :output :if-exists :supersede) @@ -125,5 +125,194 @@ (assert (= 1 (file-position s))) ; unicode... (assert (file-position s 0)))) (delete-file p)) + +;;; CLOSING a non-new streams should not delete them, and superseded +;;; files should be restored. +(let ((test "test-file-for-close-should-not-delete")) + (macrolet ((test-mode (mode) + `(progn + (catch :close-test-exit + (with-open-file (f test :direction :output :if-exists ,mode) + (write-line "test" f) + (throw :close-test-exit t))) + (assert (and (probe-file test) ,mode))))) + (unwind-protect + (progn + (with-open-file (f test :direction :output) + (write-line "test" f)) + (test-mode :append) + (test-mode :overwrite) + ;; FIXME: We really should recover supersede files as well, according to + ;; CLOSE in CLHS, but at the moment we don't. + ;; (test-mode :supersede) + (test-mode :rename) + (test-mode :rename-and-delete)) + (when (probe-file test) + (delete-file test))))) + +;;; test for read-write invariance of signed bytes, from Bruno Haible +;;; cmucl-imp 2004-09-06 +(defun bin-stream-test (&key (size (integer-length most-positive-fixnum)) + (type 'unsigned-byte) (file-name "stream-impure.tmp") + (num-bytes 10) + (bytes (if (eq type 'signed-byte) + (loop :repeat num-bytes :collect + (- (random (ash 1 size)) + (ash 1 (1- size)))) + (loop :repeat num-bytes :collect + (random (ash 1 size)))))) + (with-open-file (foo file-name :direction :output :if-exists :supersede + :element-type (list type size)) + (dolist (byte bytes) + (write-byte byte foo))) + (unwind-protect + (with-open-file (foo file-name :direction :input + :element-type (list type size)) + (list (stream-element-type foo) (file-length foo) bytes + (loop :for byte :in bytes :for nb = (read-byte foo) :collect nb + :unless (= nb byte) :do + (flet ((by-out (sz by) + (format nil "~v,'0,' ,4:b" + (+ sz (floor sz 4)) by))) + (error "~& * [(~s ~s)] ~a != ~a~%" type size + (by-out size byte) (by-out size nb)))))) + (delete-file file-name))) +(loop for size from 2 to 40 do (bin-stream-test :size size :type 'signed-byte)) + +;;; Check READ-SEQUENCE signals a TYPE-ERROR when the sequence can't +;;; contain a stream element. +;;; +;;; These tests check READ-SEQUENCE correctness, not whether the fast +;;; or slow paths are being taken for each element type. To check the +;;; fast or slow paths, trace ANSI-STREAM-READ-BYTE (slow path) and/or +;;; READ-N-BYTES: +;;; +;;; (trace sb-impl::ansi-stream-read-byte sb-impl::read-n-bytes) +;;; +;;; The order should be ANSI-STREAM-READ-BYTE, READ-N-BYTES, +;;; READ-N-BYTES, ANSI-STREAM-READ-BYTE, ANSI-STREAM-READ-BYTE. + +(let ((pathname "read-sequence.data")) + + ;; Create the binary data. + (with-open-file (stream pathname + :direction :output + :if-exists :supersede + :element-type '(unsigned-byte 8)) + (write-byte 255 stream)) + + ;; Check the slow path for generic vectors. + (let ((sequence (make-array 1))) + (with-open-file (stream pathname + :direction :input + :element-type '(unsigned-byte 8)) + (read-sequence sequence stream) + (assert (equalp sequence #(255))))) + + ;; Check the fast path works for (UNSIGNED-BYTE 8) and (SIGNED-BYTE + ;; 8) vectors. + (let ((sequence (make-array 1 :element-type '(unsigned-byte 8)))) + (with-open-file (stream pathname + :direction :input + :element-type '(unsigned-byte 8)) + (read-sequence sequence stream) + (assert (equalp sequence #(255))))) + + (let ((sequence (make-array 1 :element-type '(signed-byte 8)))) + (with-open-file (stream pathname + :direction :input + :element-type '(signed-byte 8)) + (read-sequence sequence stream) + (assert (equalp sequence #(-1))))) + + ;; Check that a TYPE-ERROR is signalled for incompatible (sequence, + ;; stream) pairs. + + (let ((sequence (make-array 1 :element-type '(signed-byte 8)))) + (with-open-file (stream pathname + :direction :input + :element-type '(unsigned-byte 8)) + (handler-case (progn + (read-sequence sequence stream) + (error "READ-SEQUENCE didn't signal an error")) + (type-error (condition) + (assert (= (type-error-datum condition) 255)) + (assert (subtypep (type-error-expected-type condition) + '(signed-byte 8))))))) + + (let ((sequence (make-array 1 :element-type '(unsigned-byte 8)))) + (with-open-file (stream pathname + :direction :input + :element-type '(signed-byte 8)) + (handler-case (progn + (read-sequence sequence stream) + (error "READ-SEQUENCE didn't signal an error")) + (type-error (condition) + (assert (= (type-error-datum condition) -1)) + (assert (subtypep (type-error-expected-type condition) + '(unsigned-byte 8)))))))) + +;;; Check WRITE-SEQUENCE signals a TYPE-ERROR when the stream can't +;;; write a sequence element. +;;; +;;; These tests check WRITE-SEQUENCE correctness, not whether the fast +;;; or slow paths are being taken for each element type. See the +;;; READ-SEQUENCE tests above for more information. +;;; +;;; (trace sb-impl::output-unsigned-byte-full-buffered sb-impl::output-signed-byte-full-buffered sb-impl::output-raw-bytes) + +(let ((pathname "write-sequence.data") + (generic-sequence (make-array 1 :initial-contents '(255))) + (unsigned-sequence (make-array 1 + :element-type '(unsigned-byte 8) + :initial-contents '(255))) + (signed-sequence (make-array 1 + :element-type '(signed-byte 8) + :initial-contents '(-1)))) + ;; Check the slow path for generic vectors. + (with-open-file (stream pathname + :direction :output + :if-exists :supersede + :element-type '(unsigned-byte 8)) + (write-sequence generic-sequence stream)) + + ;; Check the fast path for unsigned and signed vectors. + (with-open-file (stream pathname + :direction :output + :if-exists :supersede + :element-type '(unsigned-byte 8)) + (write-sequence unsigned-sequence stream)) + + (with-open-file (stream pathname + :direction :output + :if-exists :supersede + :element-type '(signed-byte 8)) + (write-sequence signed-sequence stream)) + + ;; Check a TYPE-ERROR is signalled for unsigned and signed vectors + ;; which are incompatible with the stream element type. + (with-open-file (stream pathname + :direction :output + :if-exists :supersede + :element-type '(signed-byte 8)) + (handler-case (progn + (write-sequence unsigned-sequence stream) + (error "WRITE-SEQUENCE didn't signal an error")) + (type-error (condition) + (assert (= (type-error-datum condition) 255)) + (assert (subtypep (type-error-expected-type condition) + '(signed-byte 8)))))) + + (with-open-file (stream pathname + :direction :output + :if-exists :supersede + :element-type '(unsigned-byte 8)) + (handler-case (progn + (write-sequence signed-sequence stream) + (error "WRITE-SEQUENCE didn't signal an error")) + (type-error (condition) + (assert (= (type-error-datum condition) -1)) + (assert (subtypep (type-error-expected-type condition) + '(unsigned-byte 8))))))) + ;;; success -(quit :unix-status 104)