Fix no-op require tidying issue (#360)

Closes #355.

Author: jackfirth

default-recommendations/require-and-provide-suggestions-test.rkt

@@ -127,6 +127,13 @@ test: "require tidying should move non-phase spec forms before relative paths"
 ----------------------------------------
 
 
+test: "require tidying of only non-phase spec forms should do nothing"
+----------------------------------------
+(require (only-in racket/list first)
+         (only-in racket/hash hash-union))
+----------------------------------------
+
+
 test: "require tidying shouldn't trigger when require transformers are imported and used"
 ----------------------------------------
 (require racket/require

default-recommendations/require-and-provide-suggestions.rkt

@@ -200,7 +200,8 @@
 
 
 (define (simple-spec? spec)
-  (not (equal? (parsed-simple-import-kind spec) 'other)))
+  (define kind (parsed-simple-import-kind spec))
+  (and (not (equal? kind 'other-known)) (not (equal? kind 'other-unknown))))
 
 
 (define (import-specs-all-known? specs)
@@ -212,6 +213,7 @@
   (and (sorted? specs parsed-import-spec<=>)
        (sorted? (filter simple-spec? specs) parsed-import-spec<=> #:strictly? #true)))
 
+
 (define (import-specs-tidy specs)
   (transduce specs
              (indexing parsed-simple-import-phase)