Merge pull request #62 from ktoso/disable-jvm-creating-tests-on-macos-terminap

Author: ktoso

Tests/JExtractSwiftTests/VariableImportTests.swift

@@ -41,7 +41,7 @@ final class VariableImportTests {
     )
     st.log.logLevel = .error
 
-    try await st.analyze(swiftInterfacePath: "/fake/Fake.swiftinterface", text: class_interfaceFile)
+    try st.analyze(swiftInterfacePath: "/fake/Fake.swiftinterface", text: class_interfaceFile)
 
     let identifier = "counterInt"
     let varDecl: ImportedVariable? =
@@ -59,7 +59,6 @@ final class VariableImportTests {
     }
 
     assertOutput(
-      dump: true,
       output,
       expected:
       """

Tests/JavaKitTests/BasicRuntimeTests.swift

@@ -16,14 +16,21 @@ import JavaKit
 import JavaKitNetwork
 import JavaKitVM
 import Testing
+import Foundation
+
+#if os(Linux)
+import Glibc
+#else
+import Darwin
+#endif
 
 @MainActor
 let jvm = try! JavaVirtualMachine(vmOptions: [])
 
 @Suite
 @MainActor
 struct BasicRuntimeTests {
-  @Test("Object management", .disabled(if: isLinux, "Attempts to refcount a null pointer on Linux"))
+  @Test("Object management", .disabled(if: isMacOSTerminal || isLinux, "JVM creation fails occasionally in terminal on macOS, and some issues on Linux"))
   func javaObjectManagement() throws {
     let sneakyJavaThis: jobject
     do {
@@ -49,7 +56,7 @@ struct BasicRuntimeTests {
     #expect(url.javaHolder === urlAgain.javaHolder)
   }
 
-  @Test("Java exceptions", .disabled(if: isLinux, "Attempts to refcount a null pointer on Linux"))
+  @Test("Java exceptions", .disabled(if: isMacOSTerminal || isLinux, "JVM creation fails occasionally in terminal on macOS, and some issues on Linux"))
   func javaExceptionsInSwift() throws {
     do {
       _ = try URL("bad url", environment: jvm.environment)
@@ -58,13 +65,13 @@ struct BasicRuntimeTests {
     }
   }
 
-  @Test("Static methods", .disabled(if: isMacOS, "Fails on macOS command line"))
+  @Test("Static methods", .disabled(if: isMacOSTerminal || isLinux, "JVM creation fails occasionally in terminal on macOS, and some issues on Linux"))
   func staticMethods() throws {
     let urlConnectionClass = try JavaClass<URLConnection>(in: jvm.environment)
     #expect(urlConnectionClass.getDefaultAllowUserInteraction() == false)
   }
 
-  @Test("Class instance lookup")
+  @Test("Class instance lookup", .disabled(if: isMacOSTerminal || isLinux, "JVM creation fails occasionally in terminal on macOS, and some issues on Linux"))
   func classInstanceLookup() throws {
     do {
       _ = try JavaClass<Nonexistent>(in: jvm.environment)
@@ -86,6 +93,14 @@ var isLinux: Bool {
   #endif
 }
 
+/// Whether we're running on MacOS in an interactive terminal session.
+var isMacOSTerminal: Bool {
+  isMacOS && (
+    isatty(STDOUT_FILENO) == 1 ||
+    ProcessInfo.processInfo.environment["IS_TTY"] != nil // since 'swift test' still sometimes hides the fact we're in tty
+  )
+}
+
 /// Whether we're running on MacOS.
 var isMacOS: Bool {
   #if os(macOS)