Merge pull request #50 from saviorand/feature/no-string-conversions

Improve perfomance and update to Mojo 24.4

Author: saviorand

.github/workflows/package.yml renamed to .github/workflows/main.yml

@@ -1,34 +1,42 @@
-name: Package and Release
+name: Main pipeline
 
 on:
   push:
     branches:
       - main
+
 permissions:
   contents: write
+
 jobs:
-  run-tests:
-    name: Release package
+  setup:
+    name: Setup environment and install dependencies
     runs-on: ubuntu-latest
-
     steps:
       - name: Checkout code
         uses: actions/checkout@v2
-
       - name: Install modular
         run: |
           curl -s https://get.modular.com | sh -
           modular auth examples
-
       - name: Install Mojo
         run: modular install mojo
-
       - name: Add to PATH
         run: echo "/home/runner/.modular/pkg/packages.modular.com_mojo/bin" >> $GITHUB_PATH
-
-      - name: Create package
+  test:
+    name: Run tests
+    runs-on: ubuntu-latest
+    needs: setup
+    steps:
+      - name: Run the test suite
+        run: mojo run_tests.mojo
+  package:
+    name: Create package
+    runs-on: ubuntu-latest
+    needs: setup
+    steps:
+      - name: Run the package command
         run: mojo package lightbug_http -o lightbug_http.mojopkg
-
       - name: Upload package to release
         uses: svenstaro/upload-release-action@v2
         with:

.gitignore

@@ -1,3 +1,4 @@
 *.📦
 .DS_Store
-.mojoenv
+.mojoenv
+install_id

bench.mojo

@@ -2,32 +2,25 @@ import benchmark
 from lightbug_http.sys.server import SysServer
 from lightbug_http.python.server import PythonServer
 from lightbug_http.service import TechEmpowerRouter
-from lightbug_http.tests.utils import (
+from tests.utils import (
     TestStruct,
     FakeResponder,
     new_fake_listener,
     FakeServer,
     getRequest,
 )
-from external.libc import __test_socket_client__
 
 
 fn main():
     try:
-        var server = SysServer()
+        var server = SysServer(tcp_keep_alive=True)
         var handler = TechEmpowerRouter()
         server.listen_and_serve("0.0.0.0:8080", handler)
     except e:
         print("Error starting server: " + e.__str__())
         return
 
 
-fn lightbug_benchmark_get_1req_per_conn():
-    var req_report = benchmark.run[__test_socket_client__](1, 10000, 0, 3, 100)
-    print("Request: ")
-    req_report.print(benchmark.Unit.ns)
-
-
 fn lightbug_benchmark_server():
     var server_report = benchmark.run[run_fake_server](max_iters=1)
     print("Server: ")
@@ -54,9 +47,9 @@ fn run_fake_server():
 
 fn init_test_and_set_a_copy() -> None:
     var test = TestStruct("a", "b")
-    var newtest = test.set_a_copy("c")
+    _ = test.set_a_copy("c")
 
 
 fn init_test_and_set_a_direct() -> None:
     var test = TestStruct("a", "b")
-    var newtest = test.set_a_direct("c")
+    _ = test.set_a_direct("c")

client.mojo

@@ -5,6 +5,12 @@ from lightbug_http.sys.client import MojoClient
 
 fn test_request(inout client: MojoClient) raises -> None:
     var uri = URI("http://httpbin.org/status/404")
+    try:
+        uri.parse()
+    except e:
+        print("error parsing uri: " + e.__str__())
+
+
     var request = HTTPRequest(uri)
     var response = client.do(request)
 
@@ -17,11 +23,12 @@ fn test_request(inout client: MojoClient) raises -> None:
     # print parsed headers (only some are parsed for now)
     print("Content-Type:", String(response.header.content_type()))
     print("Content-Length", response.header.content_length())
-    print("Connection:", response.header.connection_close())
     print("Server:", String(response.header.server()))
 
+    print("Is connection set to connection-close? ", response.header.connection_close())
+
     # print body
-    print(String(response.get_body()))
+    print(String(response.get_body_bytes()))
 
 
 fn main() raises -> None:

external/gojo/__init__.mojo

@@ -1 +0,0 @@
-# gojo, created by thastoasty, https://github.com/thatstoasty/gojo/