diff --git a/.swiftformat b/.swiftformat index e11af52..dd1bb2c 100644 --- a/.swiftformat +++ b/.swiftformat @@ -1,2 +1,3 @@ --swiftversion 6.2 --disable redundantMemberwiseInit +--disable swiftTestingTestCaseNames diff --git a/Sources/ForgejoKit/Models/WorkflowRun.swift b/Sources/ForgejoKit/Models/WorkflowRun.swift index f65491d..c60f4af 100644 --- a/Sources/ForgejoKit/Models/WorkflowRun.swift +++ b/Sources/ForgejoKit/Models/WorkflowRun.swift @@ -3,7 +3,7 @@ import Foundation /// Represents a single Forgejo Actions run. /// /// Mirrors the Forgejo `ActionRun` schema. Note: Forgejo merges status and -/// conclusion into a single `status` field — values are +/// conclusion into a single `status` field, values are /// `success`, `failure`, `cancelled`, `skipped`, `running`, `waiting`, /// `blocked`, `unknown`. public struct WorkflowRun: Codable, Identifiable, Sendable { diff --git a/Sources/ForgejoKit/Models/WorkflowRunView.swift b/Sources/ForgejoKit/Models/WorkflowRunView.swift index 1f787f6..1dfc0ec 100644 --- a/Sources/ForgejoKit/Models/WorkflowRunView.swift +++ b/Sources/ForgejoKit/Models/WorkflowRunView.swift @@ -3,7 +3,7 @@ import Foundation /// Response from Forgejo's internal web route /// `POST /{owner}/{repo}/actions/runs/{runIndex}/jobs/{jobIndex}`. /// -/// **Experimental.** This endpoint is the web UI's own JSON contract — it is +/// **Experimental.** This endpoint is the web UI's own JSON contract, it is /// not part of `/api/v1` and may change between Forgejo releases without /// notice. Use only for surfaces where graceful degradation is acceptable. public struct WorkflowRunView: Codable, Sendable { diff --git a/Sources/ForgejoKit/Networking/URLSessionManager.swift b/Sources/ForgejoKit/Networking/URLSessionManager.swift index aea0137..80134a9 100644 --- a/Sources/ForgejoKit/Networking/URLSessionManager.swift +++ b/Sources/ForgejoKit/Networking/URLSessionManager.swift @@ -3,7 +3,7 @@ import Foundation public class URLSessionManager: NSObject, @unchecked Sendable { public let allowSelfSignedCertificates: Bool private let trustedHost: String? - // Initialized eagerly in init — safe for concurrent access as it's only written once. + // Initialized eagerly in init, safe for concurrent access as it's only written once. private var _session: URLSession! public var session: URLSession { _session diff --git a/Sources/ForgejoKit/Services/ForgejoClient.swift b/Sources/ForgejoKit/Services/ForgejoClient.swift index a7399d8..7d75933 100644 --- a/Sources/ForgejoKit/Services/ForgejoClient.swift +++ b/Sources/ForgejoKit/Services/ForgejoClient.swift @@ -204,7 +204,7 @@ public final class ForgejoClient: Sendable { else { throw AuthenticationError.invalidResponse } return sha1 case 400, 422: - // Token name already taken — Forgejo returns 400, some versions use 422 + // Token name already taken, Forgejo returns 400, some versions use 422 let body = String(data: data, encoding: .utf8) ?? "" guard body.contains("name") else { throw AuthenticationError.from(statusCode: httpResponse.statusCode, body: body) diff --git a/Sources/ForgejoKit/Services/WorkflowService.swift b/Sources/ForgejoKit/Services/WorkflowService.swift index cbd3003..06de275 100644 --- a/Sources/ForgejoKit/Services/WorkflowService.swift +++ b/Sources/ForgejoKit/Services/WorkflowService.swift @@ -1,7 +1,7 @@ import Foundation /// Forgejo Actions API. Forgejo's Actions API is much smaller than Gitea's -/// or GitHub's — there is no public endpoint to list workflow definitions, +/// or GitHub's, there is no public endpoint to list workflow definitions, /// fetch jobs of a run, or stream job logs. Only run listing and run detail /// are exposed. public final class WorkflowService: Sendable { @@ -114,7 +114,7 @@ public final class WorkflowService: Sendable { ) // Forgejo's POST handler at this base path uses `attempt=0` which fails // for any run whose tasks are stored with the (1-based) attempt number. - // The matching GET endpoint 307-redirects to the latest-attempt URL — + // The matching GET endpoint 307-redirects to the latest-attempt URL, // resolve that first, then POST to the resolved attempt-suffixed URL. let postURL = await (try? client.discoverRedirectLocation(url: baseURL)) ?? baseURL let body = try client.encodeRequestBody(ViewRequestBody(logCursors: logCursors)) diff --git a/Tests/ForgejoKitTests/AdminServiceTests.swift b/Tests/ForgejoKitTests/AdminServiceTests.swift index 52a04a9..5620e62 100644 --- a/Tests/ForgejoKitTests/AdminServiceTests.swift +++ b/Tests/ForgejoKitTests/AdminServiceTests.swift @@ -3,7 +3,7 @@ import Foundation import Testing struct AdminServiceTests { - @Test func `create user payload encodes correct keys`() throws { + @Test func createUserPayloadEncodesCorrectKeys() throws { let json = """ { "username": "testbot", @@ -27,7 +27,7 @@ struct AdminServiceTests { #expect(dict["visibility"] as? String == "public") } - @Test func `create user URL construction`() throws { + @Test func createUserURLConstruction() throws { let client = ForgejoClient(serverURL: "https://forgejo.example.com", username: "admin", password: "pass") let url = try client.makeURL(path: "/api/v1/admin/users") #expect(url.absoluteString == "https://forgejo.example.com/api/v1/admin/users") diff --git a/Tests/ForgejoKitTests/AttachmentTests.swift b/Tests/ForgejoKitTests/AttachmentTests.swift index 2331e3b..aa005b8 100644 --- a/Tests/ForgejoKitTests/AttachmentTests.swift +++ b/Tests/ForgejoKitTests/AttachmentTests.swift @@ -11,7 +11,7 @@ struct AttachmentTests { // MARK: - Model decoding - @Test func `decodes attachment`() throws { + @Test func decodesAttachment() throws { let json = """ { "id": 1, @@ -33,8 +33,8 @@ struct AttachmentTests { #expect(attachment.isImage == true) } - @Test func `decodes attachment without type field`() throws { - // Forgejo API does not always include a type field — isImage falls back to file extension. + @Test func decodesAttachmentWithoutTypeField() throws { + // Forgejo API does not always include a type field, isImage falls back to file extension. let json = """ { "id": 3, @@ -50,7 +50,7 @@ struct AttachmentTests { #expect(attachment.isImage == true) } - @Test func `decodes non image attachment`() throws { + @Test func decodesNonImageAttachment() throws { let json = """ { "id": 2, @@ -68,7 +68,7 @@ struct AttachmentTests { #expect(attachment.isImage == false) } - @Test func `decodes issue with assets`() throws { + @Test func decodesIssueWithAssets() throws { let json = """ { "id": 1, @@ -99,7 +99,7 @@ struct AttachmentTests { #expect(issue.assets?.first?.isImage == true) } - @Test func `decodes issue with no assets`() throws { + @Test func decodesIssueWithNoAssets() throws { let json = """ { "id": 1, @@ -117,7 +117,7 @@ struct AttachmentTests { #expect(issue.assets == nil) } - @Test func `decodes issue comment with assets`() throws { + @Test func decodesIssueCommentWithAssets() throws { let json = """ { "id": 5, @@ -145,7 +145,7 @@ struct AttachmentTests { // MARK: - isImage - @Test func `is image for various mime types`() { + @Test func isImageForVariousMimeTypes() { let makeAttachment = { (type: String) in Attachment( id: 1, uuid: "u", name: "f.bin", size: 0, @@ -162,7 +162,7 @@ struct AttachmentTests { #expect(makeAttachment("application/zip").isImage == false) } - @Test func `is image falls back to file extension when type is nil`() { + @Test func isImageFallsBackToFileExtensionWhenTypeIsNil() { let makeAttachment = { (name: String) in Attachment( id: 1, uuid: "u", name: name, size: 0, @@ -180,7 +180,7 @@ struct AttachmentTests { #expect(makeAttachment("readme.txt").isImage == false) } - @Test func `is image falls back to extension when type is not a mime type`() { + @Test func isImageFallsBackToExtensionWhenTypeIsNotAMimeType() { // Forgejo returns "type": "attachment" for all uploads, not an actual MIME type. // isImage must fall through to the extension check in this case. let makeAttachment = { (name: String) in @@ -199,19 +199,19 @@ struct AttachmentTests { // MARK: - Upload URL construction - @Test func `upload issue attachment URL`() throws { + @Test func uploadIssueAttachmentURL() throws { let client = ForgejoClient(serverURL: "https://forgejo.example.com", username: "u", password: "p") let url = try client.makeRepoURL(owner: "alice", repo: "myrepo", path: "/issues/42/assets") #expect(url.absoluteString == "https://forgejo.example.com/api/v1/repos/alice/myrepo/issues/42/assets") } - @Test func `upload comment attachment URL`() throws { + @Test func uploadCommentAttachmentURL() throws { let client = ForgejoClient(serverURL: "https://forgejo.example.com", username: "u", password: "p") let url = try client.makeRepoURL(owner: "alice", repo: "myrepo", path: "/issues/comments/99/assets") #expect(url.absoluteString == "https://forgejo.example.com/api/v1/repos/alice/myrepo/issues/comments/99/assets") } - @Test func `upload URL encodes owner and repo`() throws { + @Test func uploadURLEncodesOwnerAndRepo() throws { let client = ForgejoClient(serverURL: "https://forgejo.example.com", username: "u", password: "p") let url = try client.makeRepoURL(owner: "my org", repo: "my repo", path: "/issues/1/assets") #expect(url.absoluteString == "https://forgejo.example.com/api/v1/repos/my%20org/my%20repo/issues/1/assets") @@ -219,7 +219,7 @@ struct AttachmentTests { // MARK: - Multipart body - @Test func `build multipart body structure`() { + @Test func buildMultipartBodyStructure() { let fileData = Data("hello".utf8) let boundary = "test-boundary-123" let body = ForgejoClient.buildMultipartBody( @@ -236,7 +236,7 @@ struct AttachmentTests { #expect(bodyString.hasSuffix("--\(boundary)--\r\n")) } - @Test func `build multipart body contains file data`() { + @Test func buildMultipartBodyContainsFileData() { let fileData = Data([0xFF, 0xD8, 0xFF, 0xE0]) // JPEG header bytes let boundary = "b" let body = ForgejoClient.buildMultipartBody( diff --git a/Tests/ForgejoKitTests/BearerCredentialTests.swift b/Tests/ForgejoKitTests/BearerCredentialTests.swift index e7d887a..a612a12 100644 --- a/Tests/ForgejoKitTests/BearerCredentialTests.swift +++ b/Tests/ForgejoKitTests/BearerCredentialTests.swift @@ -3,21 +3,21 @@ import Foundation import Testing struct BearerCredentialTests { - @Test func `bearer credential uses bearer scheme`() { + @Test func bearerCredentialUsesBearerScheme() { let client = ForgejoClient( serverURL: "https://codeberg.org", username: "octocat", bearerToken: "abc123", ) #expect(client.authorizationHeaderValue == "Bearer abc123") } - @Test func `token credential still uses token scheme`() { + @Test func tokenCredentialStillUsesTokenScheme() { let client = ForgejoClient( serverURL: "https://codeberg.org", username: "octocat", token: "abc123", ) #expect(client.authorizationHeaderValue == "token abc123") } - @Test func `basic credential uses basic scheme`() { + @Test func basicCredentialUsesBasicScheme() { let client = ForgejoClient( serverURL: "https://codeberg.org", username: "octocat", password: "pw", ) diff --git a/Tests/ForgejoKitTests/DateDecodingTests.swift b/Tests/ForgejoKitTests/DateDecodingTests.swift index cbc13c4..8798900 100644 --- a/Tests/ForgejoKitTests/DateDecodingTests.swift +++ b/Tests/ForgejoKitTests/DateDecodingTests.swift @@ -21,7 +21,7 @@ struct DateDecodingTests { // MARK: - ISO 8601 without fractional seconds - @Test func `decodes date without fractional seconds`() throws { + @Test func decodesDateWithoutFractionalSeconds() throws { let json = #"{"date":"2024-01-15T10:30:00Z"}"# let result = try decoder().decode(DateWrapper.self, from: Data(json.utf8)) @@ -34,7 +34,7 @@ struct DateDecodingTests { // MARK: - ISO 8601 with fractional seconds - @Test func `decodes date with fractional seconds`() throws { + @Test func decodesDateWithFractionalSeconds() throws { let json = #"{"date":"2024-06-20T14:05:30.123Z"}"# let result = try decoder().decode(DateWrapper.self, from: Data(json.utf8)) @@ -44,7 +44,7 @@ struct DateDecodingTests { #expect(utcCalendar.component(.hour, from: result.date) == 14) } - @Test func `decodes date with triple zero fraction`() throws { + @Test func decodesDateWithTripleZeroFraction() throws { let json = #"{"date":"2024-01-15T10:30:00.000Z"}"# let result = try decoder().decode(DateWrapper.self, from: Data(json.utf8)) @@ -53,14 +53,14 @@ struct DateDecodingTests { // MARK: - Invalid dates - @Test func `throws on invalid date string`() { + @Test func throwsOnInvalidDateString() { let json = #"{"date":"not-a-date"}"# #expect(throws: DecodingError.self) { _ = try decoder().decode(DateWrapper.self, from: Data(json.utf8)) } } - @Test func `throws on empty date string`() { + @Test func throwsOnEmptyDateString() { let json = #"{"date":""}"# #expect(throws: DecodingError.self) { _ = try decoder().decode(DateWrapper.self, from: Data(json.utf8)) @@ -69,7 +69,7 @@ struct DateDecodingTests { // MARK: - Both formats decode to same value - @Test func `both formats produce same date`() throws { + @Test func bothFormatsProduceSameDate() throws { let jsonWithout = #"{"date":"2024-03-10T08:00:00Z"}"# let jsonWith = #"{"date":"2024-03-10T08:00:00.000Z"}"# diff --git a/Tests/ForgejoKitTests/DiffParserTests.swift b/Tests/ForgejoKitTests/DiffParserTests.swift index 2864056..5e896e2 100644 --- a/Tests/ForgejoKitTests/DiffParserTests.swift +++ b/Tests/ForgejoKitTests/DiffParserTests.swift @@ -5,19 +5,19 @@ import Testing struct DiffParserTests { // MARK: - Empty and minimal input - @Test func `parse empty string`() { + @Test func parseEmptyString() { let result = DiffParser.parse("") #expect(result.files.isEmpty) } - @Test func `parse garbage input`() { + @Test func parseGarbageInput() { let result = DiffParser.parse("this is not a diff") #expect(result.files.isEmpty) } // MARK: - Single file diff - @Test func `parse single file addition`() { + @Test func parseSingleFileAddition() { let diff = [ "diff --git a/hello.txt b/hello.txt", "new file mode 100644", @@ -45,7 +45,7 @@ struct DiffParserTests { #expect(hunk.lines[3].type == .addition) } - @Test func `parse single file deletion`() { + @Test func parseSingleFileDeletion() { let diff = [ "diff --git a/old.txt b/old.txt", "deleted file mode 100644", @@ -64,7 +64,7 @@ struct DiffParserTests { #expect(file.newName == "/dev/null") } - @Test func `parse single file modification`() { + @Test func parseSingleFileModification() { let diff = [ "diff --git a/file.txt b/file.txt", "--- a/file.txt", @@ -107,7 +107,7 @@ struct DiffParserTests { // MARK: - Multiple files - @Test func `parse multiple files`() { + @Test func parseMultipleFiles() { let diff = [ "diff --git a/a.txt b/a.txt", "--- a/a.txt", @@ -131,7 +131,7 @@ struct DiffParserTests { // MARK: - Multiple hunks - @Test func `parse multiple hunks`() { + @Test func parseMultipleHunks() { let diff = [ "diff --git a/file.txt b/file.txt", "--- a/file.txt", @@ -159,7 +159,7 @@ struct DiffParserTests { // MARK: - Line number tracking - @Test func `line numbers track correctly with multiple additions`() { + @Test func lineNumbersTrackCorrectlyWithMultipleAdditions() { let diff = [ "diff --git a/file.txt b/file.txt", "--- a/file.txt", @@ -207,7 +207,7 @@ struct DiffParserTests { // MARK: - Hunk header parsing - @Test func `hunk header with function context`() { + @Test func hunkHeaderWithFunctionContext() { let diff = [ "diff --git a/main.swift b/main.swift", "--- a/main.swift", @@ -226,7 +226,7 @@ struct DiffParserTests { // MARK: - File name extraction - @Test func `file name extracted from minus plus`() { + @Test func fileNameExtractedFromMinusPlus() { let diff = [ "diff --git a/src/app.swift b/src/app.swift", "--- a/src/app.swift", @@ -241,7 +241,7 @@ struct DiffParserTests { #expect(result.files[0].newName == "src/app.swift") } - @Test func `file name falls back to git header`() { + @Test func fileNameFallsBackToGitHeader() { let diff = [ "diff --git a/image.png b/image.png", "Binary files differ", @@ -255,7 +255,7 @@ struct DiffParserTests { // MARK: - Commit diff (new file from Forgejo API) - @Test func `parse commit diff new file`() { + @Test func parseCommitDiffNewFile() { let diff = [ "diff --git a/src/main.py b/src/main.py", "new file mode 100644", @@ -281,7 +281,7 @@ struct DiffParserTests { #expect(hunk.lines[1].oldLineNumber == nil) } - @Test func `parse commit diff multiple files`() { + @Test func parseCommitDiffMultipleFiles() { let diff = [ "diff --git a/hello.py b/hello.py", "new file mode 100644", @@ -315,7 +315,7 @@ struct DiffParserTests { // MARK: - No newline at end of file sentinel - @Test func `no newline at end of file sentinel is skipped`() { + @Test func noNewlineAtEndOfFileSentinelIsSkipped() { let diff = [ "diff --git a/file.txt b/file.txt", "--- a/file.txt", @@ -344,7 +344,7 @@ struct DiffParserTests { // MARK: - DiffLine identity - @Test func `each line has unique id`() { + @Test func eachLineHasUniqueId() { let diff = [ "diff --git a/f.txt b/f.txt", "--- a/f.txt", diff --git a/Tests/ForgejoKitTests/ErrorMappingTests.swift b/Tests/ForgejoKitTests/ErrorMappingTests.swift index 0288505..f96c3ad 100644 --- a/Tests/ForgejoKitTests/ErrorMappingTests.swift +++ b/Tests/ForgejoKitTests/ErrorMappingTests.swift @@ -2,7 +2,7 @@ import Testing struct ErrorMappingTests { - @Test func `classifies HTTP status codes`() { + @Test func classifiesHTTPStatusCodes() { #expect(HTTPErrorCategory(statusCode: 401) == .authentication) #expect(HTTPErrorCategory(statusCode: 403) == .permissionDenied) #expect(HTTPErrorCategory(statusCode: 404) == .notFound) @@ -11,7 +11,7 @@ struct ErrorMappingTests { #expect(HTTPErrorCategory(statusCode: 400) == .other) } - @Test func `exposes service error status and category`() { + @Test func exposesServiceErrorStatusAndCategory() { let forbidden = ServiceError.httpError(statusCode: 403, message: "missing scope") #expect(forbidden.httpStatusCode == 403) #expect(forbidden.httpErrorCategory == .permissionDenied) @@ -24,7 +24,7 @@ struct ErrorMappingTests { #expect(ServiceError.invalidURL.httpErrorCategory == nil) } - @Test func `maps authentication status codes`() { + @Test func mapsAuthenticationStatusCodes() { #expect(AuthenticationError.from(statusCode: 401) == .invalidCredentials) #expect(AuthenticationError.from(statusCode: 401, body: "OTP required") == .otpRequired) #expect(AuthenticationError.from(statusCode: 403) == .unknownError(statusCode: 403)) @@ -33,7 +33,7 @@ struct ErrorMappingTests { #expect(AuthenticationError.from(statusCode: 418) == .unknownError(statusCode: 418)) } - @Test func `exposes authentication status and category`() { + @Test func exposesAuthenticationStatusAndCategory() { #expect(AuthenticationError.invalidCredentials.httpStatusCode == 401) #expect(AuthenticationError.invalidCredentials.httpErrorCategory == .authentication) #expect(AuthenticationError.serverNotFound.httpStatusCode == 404) @@ -47,7 +47,7 @@ struct ErrorMappingTests { // MARK: - ServiceError descriptions - @Test func `service error descriptions`() { + @Test func serviceErrorDescriptions() { #expect(ServiceError.noActiveInstance.errorDescription == "No active Forgejo instance") #expect(ServiceError.invalidURL.errorDescription == "Invalid URL") #expect(ServiceError.invalidResponse.errorDescription == "Invalid response from server") @@ -56,7 +56,7 @@ struct ErrorMappingTests { #expect(ServiceError.decodingFailed(detail: "missing key").errorDescription == "Decoding failed: missing key") } - @Test func `http error description uses forgejo message body`() { + @Test func httpErrorDescriptionUsesForgejoMessageBody() { let error = ServiceError.httpError( statusCode: 405, message: #"{"message":"This pull request has failing status checks"}"#, @@ -65,19 +65,19 @@ struct ErrorMappingTests { #expect(error.errorDescription == "HTTP 405: This pull request has failing status checks") } - @Test func `http error description falls back to raw body`() { + @Test func httpErrorDescriptionFallsBackToRawBody() { let error = ServiceError.httpError(statusCode: 500, message: "database unavailable") #expect(error.errorDescription == "HTTP 500: database unavailable") } - @Test func `http error description ignores whitespace only message`() { + @Test func httpErrorDescriptionIgnoresWhitespaceOnlyMessage() { let error = ServiceError.httpError(statusCode: 503, message: " \n ") #expect(error.errorDescription == "HTTP error: 503") } - @Test func `http error description falls back when JSON message field is empty`() { + @Test func httpErrorDescriptionFallsBackWhenJSONMessageFieldIsEmpty() { let error = ServiceError.httpError( statusCode: 422, message: #"{"message":" "}"#, @@ -86,7 +86,7 @@ struct ErrorMappingTests { #expect(error.errorDescription == "HTTP error: 422") } - @Test func `http error description returns raw JSON when message field missing`() { + @Test func httpErrorDescriptionReturnsRawJSONWhenMessageFieldMissing() { let raw = #"{"errors":["bad request"]}"# let error = ServiceError.httpError(statusCode: 400, message: raw) @@ -95,21 +95,21 @@ struct ErrorMappingTests { // MARK: - Merge error collapsing - @Test func `collapse merge error maps 405 with empty body to not mergeable`() { + @Test func collapseMergeErrorMaps405WithEmptyBodyToNotMergeable() { let collapsed = PullRequestService.collapseMergeError( .httpError(statusCode: 405, message: ""), ) #expect(collapsed == .notMergeable) } - @Test func `collapse merge error maps 409 with empty body to merge conflict`() { + @Test func collapseMergeErrorMaps409WithEmptyBodyToMergeConflict() { let collapsed = PullRequestService.collapseMergeError( .httpError(statusCode: 409, message: " \n "), ) #expect(collapsed == .mergeConflict) } - @Test func `collapse merge error preserves 405 with body`() { + @Test func collapseMergeErrorPreserves405WithBody() { let original = ServiceError.httpError( statusCode: 405, message: #"{"message":"failing status checks"}"#, @@ -117,17 +117,17 @@ struct ErrorMappingTests { #expect(PullRequestService.collapseMergeError(original) == original) } - @Test func `collapse merge error preserves 409 with body`() { + @Test func collapseMergeErrorPreserves409WithBody() { let original = ServiceError.httpError(statusCode: 409, message: "conflict in README.md") #expect(PullRequestService.collapseMergeError(original) == original) } - @Test func `collapse merge error passes other status codes through`() { + @Test func collapseMergeErrorPassesOtherStatusCodesThrough() { let original = ServiceError.httpError(statusCode: 500, message: "") #expect(PullRequestService.collapseMergeError(original) == original) } - @Test func `collapse merge error passes non HTTP errors through`() { + @Test func collapseMergeErrorPassesNonHTTPErrorsThrough() { #expect(PullRequestService.collapseMergeError(.invalidURL) == .invalidURL) #expect(PullRequestService.collapseMergeError(.noActiveInstance) == .noActiveInstance) } diff --git a/Tests/ForgejoKitTests/FileContentTests.swift b/Tests/ForgejoKitTests/FileContentTests.swift index ad6c406..317d846 100644 --- a/Tests/ForgejoKitTests/FileContentTests.swift +++ b/Tests/ForgejoKitTests/FileContentTests.swift @@ -21,14 +21,14 @@ struct FileContentTests { // MARK: - Base64 decoding - @Test func `decodes base 64 content`() { + @Test func decodesBase64Content() { let original = "Hello, World!" let base64 = Data(original.utf8).base64EncodedString() let file = makeFileContent(content: base64, encoding: "base64") #expect(file.decodedContent == "Hello, World!") } - @Test func `decodes base 64 with newlines`() { + @Test func decodesBase64WithNewlines() { let original = "Line one\nLine two\nLine three" let base64 = Data(original.utf8).base64EncodedString() let wrappedBase64 = String(base64.prefix(20)) + "\n" + String(base64.dropFirst(20)) @@ -36,29 +36,29 @@ struct FileContentTests { #expect(file.decodedContent == original) } - @Test func `returns nil content when content is nil`() { + @Test func returnsNilContentWhenContentIsNil() { let file = makeFileContent(content: nil, encoding: "base64") #expect(file.decodedContent == nil) } - @Test func `returns raw content when encoding is nil`() { + @Test func returnsRawContentWhenEncodingIsNil() { let file = makeFileContent(content: "raw text", encoding: nil) #expect(file.decodedContent == "raw text") } - @Test func `returns raw content when encoding is not base 64`() { + @Test func returnsRawContentWhenEncodingIsNotBase64() { let file = makeFileContent(content: "raw text", encoding: "utf-8") #expect(file.decodedContent == "raw text") } - @Test func `returns nil for invalid base 64`() { + @Test func returnsNilForInvalidBase64() { let file = makeFileContent(content: "!!!not-base64!!!", encoding: "base64") #expect(file.decodedContent == nil) } // MARK: - JSON decoding - @Test func `decodes file content from JSON`() throws { + @Test func decodesFileContentFromJSON() throws { let json = """ { "name": "readme.md", diff --git a/Tests/ForgejoKitTests/ModelDecodingTests.swift b/Tests/ForgejoKitTests/ModelDecodingTests.swift index 86acc23..8724804 100644 --- a/Tests/ForgejoKitTests/ModelDecodingTests.swift +++ b/Tests/ForgejoKitTests/ModelDecodingTests.swift @@ -11,7 +11,7 @@ struct ModelDecodingTests { // MARK: - User - @Test func `decodes user`() throws { + @Test func decodesUser() throws { let json = """ { "id": 1, @@ -31,7 +31,7 @@ struct ModelDecodingTests { #expect(user.isAdmin == true) } - @Test func `decodes user with minimal fields`() throws { + @Test func decodesUserWithMinimalFields() throws { let json = """ { "id": 2, @@ -47,7 +47,7 @@ struct ModelDecodingTests { // MARK: - IssueLabel - @Test func `decodes issue label`() throws { + @Test func decodesIssueLabel() throws { let json = """ { "id": 10, @@ -64,7 +64,7 @@ struct ModelDecodingTests { // MARK: - IssueComment - @Test func `decodes issue comment`() throws { + @Test func decodesIssueComment() throws { let json = """ { "id": 100, @@ -82,7 +82,7 @@ struct ModelDecodingTests { // MARK: - Issue - @Test func `decodes issue`() throws { + @Test func decodesIssue() throws { let json = """ { "id": 50, @@ -107,7 +107,7 @@ struct ModelDecodingTests { #expect(issue.repository == nil) } - @Test func `decodes issue with milestone and assignees`() throws { + @Test func decodesIssueWithMilestoneAndAssignees() throws { let json = """ { "id": 56, @@ -147,7 +147,7 @@ struct ModelDecodingTests { #expect(issue.assignees?[1].login == "dev2") } - @Test func `decodes issue with nil milestone and empty assignees`() throws { + @Test func decodesIssueWithNilMilestoneAndEmptyAssignees() throws { let json = """ { "id": 57, @@ -168,7 +168,7 @@ struct ModelDecodingTests { #expect(issue.assignees?.isEmpty == true) } - @Test func `decodes issue milestone`() throws { + @Test func decodesIssueMilestone() throws { let json = """ { "id": 1, @@ -188,7 +188,7 @@ struct ModelDecodingTests { #expect(milestone.closedAt == nil) } - @Test func `decodes issue milestone with minimal fields`() throws { + @Test func decodesIssueMilestoneWithMinimalFields() throws { let json = """ { "id": 2, @@ -204,7 +204,7 @@ struct ModelDecodingTests { #expect(milestone.closedAt == nil) } - @Test func `decodes issue with minimal repository`() throws { + @Test func decodesIssueWithMinimalRepository() throws { let json = """ { "id": 52, @@ -233,7 +233,7 @@ struct ModelDecodingTests { #expect(issue.repository?.defaultBranch == nil) } - @Test func `decodes issue with repository`() throws { + @Test func decodesIssueWithRepository() throws { let json = """ { "id": 51, @@ -284,7 +284,7 @@ struct ModelDecodingTests { #expect(issue.repository?.fullName == "user/my-repo") } - @Test func `decodes issue with merged pull request field`() throws { + @Test func decodesIssueWithMergedPullRequestField() throws { let json = """ { "id": 53, @@ -315,7 +315,7 @@ struct ModelDecodingTests { #expect(issue.repository?.fullName == "org/app") } - @Test func `decodes issue with open pull request field`() throws { + @Test func decodesIssueWithOpenPullRequestField() throws { let json = """ { "id": 54, @@ -346,7 +346,7 @@ struct ModelDecodingTests { #expect(issue.comments == 0) } - @Test func `decodes issue with minimal pull request field`() throws { + @Test func decodesIssueWithMinimalPullRequestField() throws { let json = """ { "id": 55, @@ -371,7 +371,7 @@ struct ModelDecodingTests { // MARK: - PullRequest - @Test func `decodes pull request`() throws { + @Test func decodesPullRequest() throws { let json = """ { "id": 200, @@ -409,7 +409,7 @@ struct ModelDecodingTests { #expect(pullRequest.draft == false) } - @Test func `decodes pull request with full metadata`() throws { + @Test func decodesPullRequestWithFullMetadata() throws { let json = """ { "id": 204, @@ -460,7 +460,7 @@ struct ModelDecodingTests { #expect(pullRequest.requestedReviewers?[0].login == "reviewer1") } - @Test func `decodes pull request with requested reviewers`() throws { + @Test func decodesPullRequestWithRequestedReviewers() throws { let json = """ { "id": 202, @@ -490,7 +490,7 @@ struct ModelDecodingTests { #expect(pullRequest.requestedReviewers?[1].fullName == "Reviewer Two") } - @Test func `decodes pull request without requested reviewers`() throws { + @Test func decodesPullRequestWithoutRequestedReviewers() throws { let json = """ { "id": 203, @@ -511,7 +511,7 @@ struct ModelDecodingTests { #expect(pullRequest.requestedReviewers == nil) } - @Test func `decodes merged pull request`() throws { + @Test func decodesMergedPullRequest() throws { let json = """ { "id": 201, @@ -538,7 +538,7 @@ struct ModelDecodingTests { // MARK: - PullRequestReview - @Test func `decodes pull request review`() throws { + @Test func decodesPullRequestReview() throws { let json = """ { "id": 300, @@ -562,7 +562,7 @@ struct ModelDecodingTests { #expect(review.official == true) } - @Test func `decodes review with nullable fields`() throws { + @Test func decodesReviewWithNullableFields() throws { let json = """ { "id": 301, @@ -578,7 +578,7 @@ struct ModelDecodingTests { // MARK: - ReviewComment - @Test func `decodes review comment`() throws { + @Test func decodesReviewComment() throws { let json = """ { "id": 400, @@ -601,7 +601,7 @@ struct ModelDecodingTests { // MARK: - CreateReviewComment encoding - @Test func `encodes create review comment`() throws { + @Test func encodesCreateReviewComment() throws { let comment = CreateReviewComment( body: "Fix this", path: "file.swift", @@ -616,7 +616,7 @@ struct ModelDecodingTests { #expect(dict["new_position"] as? Int == 7) } - @Test func `encodes create review comment with nil positions`() throws { + @Test func encodesCreateReviewCommentWithNilPositions() throws { let comment = CreateReviewComment( body: "Note", path: "readme.md", @@ -631,7 +631,7 @@ struct ModelDecodingTests { // MARK: - Repository - @Test func `decodes repository`() throws { + @Test func decodesRepository() throws { let json = """ { "id": 1, @@ -677,7 +677,7 @@ struct ModelDecodingTests { #expect(repo.updatedAt != nil) } - @Test func `decodes repository with minimal fields`() throws { + @Test func decodesRepositoryWithMinimalFields() throws { let json = """ { "id": 10, @@ -713,7 +713,7 @@ struct ModelDecodingTests { #expect(repo.template == nil) } - @Test func `decodes repository with partial fields`() throws { + @Test func decodesRepositoryWithPartialFields() throws { let json = """ { "id": 11, @@ -738,7 +738,7 @@ struct ModelDecodingTests { // MARK: - NotificationThread - @Test func `decodes notification thread`() throws { + @Test func decodesNotificationThread() throws { let json = """ { "id": 1, @@ -775,7 +775,7 @@ struct ModelDecodingTests { #expect(notification.repository.fullName == "org/app") } - @Test func `decodes notification subject types`() throws { + @Test func decodesNotificationSubjectTypes() throws { let issueJson = """ { "id": 2, @@ -832,7 +832,7 @@ struct ModelDecodingTests { #expect(commit.subject.state == nil) } - @Test func `decodes notification count`() throws { + @Test func decodesNotificationCount() throws { let json = """ {"new": 5} """ @@ -840,7 +840,7 @@ struct ModelDecodingTests { #expect(count.new == 5) } - @Test func `decodes notification with minimal fields`() throws { + @Test func decodesNotificationWithMinimalFields() throws { let json = """ { "id": 10, @@ -867,7 +867,7 @@ struct ModelDecodingTests { // MARK: - Commit - @Test func `decodes commit`() throws { + @Test func decodesCommit() throws { let json = """ { "sha": "abc123def456", @@ -913,7 +913,7 @@ struct ModelDecodingTests { #expect(commit.id == "abc123def456") } - @Test func `decodes commit with minimal fields`() throws { + @Test func decodesCommitWithMinimalFields() throws { let json = """ { "sha": "minimal123", @@ -934,7 +934,7 @@ struct ModelDecodingTests { #expect(commit.url == nil) } - @Test func `decodes commit with null author`() throws { + @Test func decodesCommitWithNullAuthor() throws { let json = """ { "sha": "noauthor456", @@ -959,7 +959,7 @@ struct ModelDecodingTests { #expect(commit.parents?.isEmpty == true) } - @Test func `decodes commit list`() throws { + @Test func decodesCommitList() throws { let json = """ [ { @@ -983,7 +983,7 @@ struct ModelDecodingTests { // MARK: - RepositoryContent - @Test func `decodes repository content`() throws { + @Test func decodesRepositoryContent() throws { let json = """ { "name": "README.md", @@ -1002,7 +1002,7 @@ struct ModelDecodingTests { #expect(content.id == "README.md") } - @Test func `decodes directory content`() throws { + @Test func decodesDirectoryContent() throws { let json = """ { "name": "src", @@ -1021,7 +1021,7 @@ struct ModelDecodingTests { // MARK: - Branch - @Test func `decodes branch`() throws { + @Test func decodesBranch() throws { let json = """ { "name": "main", @@ -1042,7 +1042,7 @@ struct ModelDecodingTests { // MARK: - Repository Equatable / Hashable - @Test func `repositories with same id are equal`() throws { + @Test func repositoriesWithSameIdAreEqual() throws { let json1 = """ {"id": 42, "name": "repo-a", "full_name": "owner/repo-a", "owner": {"id": 1, "login": "owner"}, "html_url": "https://example.com/owner/repo-a", "description": "", "empty": false, "private": false, "fork": false, "mirror": false, "archived": false, "stars_count": 0, "forks_count": 0, "open_issues_count": 0, "open_pr_counter": 0, "has_issues": true, "has_pull_requests": true, "default_branch": "main", "created_at": "2024-01-01T00:00:00Z", "updated_at": "2024-01-01T00:00:00Z"} """ @@ -1055,7 +1055,7 @@ struct ModelDecodingTests { #expect(repo1.hashValue == repo2.hashValue) } - @Test func `repositories with different ids are not equal`() throws { + @Test func repositoriesWithDifferentIdsAreNotEqual() throws { let json1 = """ {"id": 1, "name": "repo", "full_name": "owner/repo", "owner": {"id": 1, "login": "owner"}, "html_url": "https://example.com/owner/repo", "description": "", "empty": false, "private": false, "fork": false, "mirror": false, "archived": false, "stars_count": 0, "forks_count": 0, "open_issues_count": 0, "open_pr_counter": 0, "has_issues": true, "has_pull_requests": true, "default_branch": "main", "created_at": "2024-01-01T00:00:00Z", "updated_at": "2024-01-01T00:00:00Z"} """ @@ -1069,7 +1069,7 @@ struct ModelDecodingTests { // MARK: - RepositoryPermissions - @Test func `decodes repository with full permissions`() throws { + @Test func decodesRepositoryWithFullPermissions() throws { let json = """ { "id": 20, @@ -1088,7 +1088,7 @@ struct ModelDecodingTests { #expect(repo.permissions?.pull == true) } - @Test func `decodes repository with read only permissions`() throws { + @Test func decodesRepositoryWithReadOnlyPermissions() throws { let json = """ { "id": 21, @@ -1107,7 +1107,7 @@ struct ModelDecodingTests { #expect(repo.permissions?.pull == true) } - @Test func `decodes repository with missing permissions`() throws { + @Test func decodesRepositoryWithMissingPermissions() throws { let json = """ { "id": 22, @@ -1121,7 +1121,7 @@ struct ModelDecodingTests { // MARK: - Token - @Test func `decodes token`() throws { + @Test func decodesToken() throws { let json = """ { "id": 1, @@ -1137,7 +1137,7 @@ struct ModelDecodingTests { #expect(token.tokenLastEight == "56789abc") } - @Test func `decodes token with minimal fields`() throws { + @Test func decodesTokenWithMinimalFields() throws { let json = """ { "id": 2, @@ -1154,7 +1154,7 @@ struct ModelDecodingTests { // MARK: - WorkflowRun (Forgejo's ActionRun schema) - @Test func `decodes workflow run with full fields`() throws { + @Test func decodesWorkflowRunWithFullFields() throws { let json = """ { "id": 100, @@ -1194,7 +1194,7 @@ struct ModelDecodingTests { #expect(run.durationNanos == 240_000_000_000) } - @Test func `decodes workflow run with minimal fields`() throws { + @Test func decodesWorkflowRunWithMinimalFields() throws { // Minimum the Forgejo API guarantees for an in-flight run: id and status. let json = """ { @@ -1212,7 +1212,7 @@ struct ModelDecodingTests { // MARK: - WorkflowRunView (experimental web-route response) - @Test func `decodes workflow run view`() throws { + @Test func decodesWorkflowRunView() throws { let json = """ { "state": { @@ -1257,7 +1257,7 @@ struct ModelDecodingTests { #expect(view.logs.stepsLog.first?.lines.first?.message == "hello") } - @Test func `decodes workflow run view with empty logs`() throws { + @Test func decodesWorkflowRunViewWithEmptyLogs() throws { let json = """ { "state": { @@ -1284,7 +1284,7 @@ struct ModelDecodingTests { // MARK: - DecodingError description - @Test func `decoding error at root shows root`() { + @Test func decodingErrorAtRootShowsRoot() { // Decoding a String where an object is expected produces a root-level error let json = #""not-an-object""# do { diff --git a/Tests/ForgejoKitTests/NormalizeServerURLTests.swift b/Tests/ForgejoKitTests/NormalizeServerURLTests.swift index 4bd0cda..8f2c33b 100644 --- a/Tests/ForgejoKitTests/NormalizeServerURLTests.swift +++ b/Tests/ForgejoKitTests/NormalizeServerURLTests.swift @@ -3,47 +3,47 @@ import Foundation import Testing struct NormalizeServerURLTests { - @Test func `strips trailing slash`() { + @Test func stripsTrailingSlash() { let result = ForgejoClient.normalizeServerURL("https://forgejo.example.com/") #expect(result == "https://forgejo.example.com") } - @Test func `adds https when no scheme`() { + @Test func addsHttpsWhenNoScheme() { let result = ForgejoClient.normalizeServerURL("forgejo.example.com") #expect(result == "https://forgejo.example.com") } - @Test func `preserves http scheme`() { + @Test func preservesHttpScheme() { let result = ForgejoClient.normalizeServerURL("http://localhost:3000") #expect(result == "http://localhost:3000") } - @Test func `preserves https scheme`() { + @Test func preservesHttpsScheme() { let result = ForgejoClient.normalizeServerURL("https://forgejo.example.com") #expect(result == "https://forgejo.example.com") } - @Test func `trims whitespace`() { + @Test func trimsWhitespace() { let result = ForgejoClient.normalizeServerURL(" https://forgejo.example.com ") #expect(result == "https://forgejo.example.com") } - @Test func `handles trailing slash and whitespace`() { + @Test func handlesTrailingSlashAndWhitespace() { let result = ForgejoClient.normalizeServerURL(" forgejo.example.com/ ") #expect(result == "https://forgejo.example.com") } - @Test func `preserves port`() { + @Test func preservesPort() { let result = ForgejoClient.normalizeServerURL("https://forgejo.example.com:3000") #expect(result == "https://forgejo.example.com:3000") } - @Test func `preserves path`() { + @Test func preservesPath() { let result = ForgejoClient.normalizeServerURL("https://example.com/forgejo") #expect(result == "https://example.com/forgejo") } - @Test func `strips multiple trailing slashes`() { + @Test func stripsMultipleTrailingSlashes() { let result = ForgejoClient.normalizeServerURL("https://example.com///") #expect(result == "https://example.com") } diff --git a/Tests/ForgejoKitTests/NotificationTests.swift b/Tests/ForgejoKitTests/NotificationTests.swift index bb81cb9..a3af03c 100644 --- a/Tests/ForgejoKitTests/NotificationTests.swift +++ b/Tests/ForgejoKitTests/NotificationTests.swift @@ -11,61 +11,61 @@ struct NotificationTests { // MARK: - Subject number extraction - @Test func `extracts issue number from subject url`() { + @Test func extractsIssueNumberFromSubjectUrl() { let url = "https://forgejo.example.com/api/v1/repos/org/app/issues/42" #expect(notificationSubjectNumber(from: url) == 42) } - @Test func `extracts pull request number from subject url`() { + @Test func extractsPullRequestNumberFromSubjectUrl() { let url = "https://forgejo.example.com/api/v1/repos/user/repo/pulls/7" #expect(notificationSubjectNumber(from: url) == 7) } - @Test func `returns nil for nil url`() { + @Test func returnsNilForNilUrl() { #expect(notificationSubjectNumber(from: nil) == nil) } - @Test func `returns nil for non numeric last component`() { + @Test func returnsNilForNonNumericLastComponent() { let url = "https://forgejo.example.com/api/v1/repos/org/app/commits/abc123" #expect(notificationSubjectNumber(from: url) == nil) } - @Test func `returns nil for empty string`() { + @Test func returnsNilForEmptyString() { #expect(notificationSubjectNumber(from: "") == nil) } - @Test func `returns nil for invalid url`() { + @Test func returnsNilForInvalidUrl() { #expect(notificationSubjectNumber(from: "not a url at all %%%") == nil) } - @Test func `extracts number from url with trailing slash`() { + @Test func extractsNumberFromUrlWithTrailingSlash() { let url = "https://forgejo.example.com/api/v1/repos/org/app/issues/42/" #expect(notificationSubjectNumber(from: url) == 42) } - @Test func `extracts large number`() { + @Test func extractsLargeNumber() { let url = "https://forgejo.example.com/api/v1/repos/org/app/issues/99999" #expect(notificationSubjectNumber(from: url) == 99999) } - @Test func `extracts number from url with query params`() { + @Test func extractsNumberFromUrlWithQueryParams() { let url = "https://forgejo.example.com/api/v1/repos/org/app/issues/42?foo=bar&baz=1" #expect(notificationSubjectNumber(from: url) == 42) } - @Test func `extracts number from url with fragment`() { + @Test func extractsNumberFromUrlWithFragment() { let url = "https://forgejo.example.com/api/v1/repos/org/app/pulls/7#section" #expect(notificationSubjectNumber(from: url) == 7) } - @Test func `extracts number from url with query and fragment`() { + @Test func extractsNumberFromUrlWithQueryAndFragment() { let url = "https://forgejo.example.com/api/v1/repos/org/app/issues/15?ref=main#comment-1" #expect(notificationSubjectNumber(from: url) == 15) } // MARK: - Full API response decoding - @Test func `decodes notification array`() throws { + @Test func decodesNotificationArray() throws { let json = """ [ { @@ -106,19 +106,19 @@ struct NotificationTests { #expect(notifications[1].subject.type == "Pull") } - @Test func `decodes empty notification array`() throws { + @Test func decodesEmptyNotificationArray() throws { let json = "[]" let notifications = try decoder().decode([NotificationThread].self, from: Data(json.utf8)) #expect(notifications.isEmpty) } - @Test func `decodes notification count zero`() throws { + @Test func decodesNotificationCountZero() throws { let json = #"{"new": 0}"# let count = try decoder().decode(NotificationCount.self, from: Data(json.utf8)) #expect(count.new == 0) } - @Test func `decodes notification count large value`() throws { + @Test func decodesNotificationCountLargeValue() throws { let json = #"{"new": 9999}"# let count = try decoder().decode(NotificationCount.self, from: Data(json.utf8)) #expect(count.new == 9999) @@ -126,7 +126,7 @@ struct NotificationTests { // MARK: - Date handling in notifications - @Test func `decodes notification with fractional seconds date`() throws { + @Test func decodesNotificationWithFractionalSecondsDate() throws { let json = """ { "id": 5, @@ -144,7 +144,7 @@ struct NotificationTests { // MARK: - Extra/unknown fields are ignored - @Test func `decodes notification ignoring extra fields`() throws { + @Test func decodesNotificationIgnoringExtraFields() throws { let json = """ { "id": 6, @@ -167,7 +167,7 @@ struct NotificationTests { // MARK: - Required fields validation - @Test func `fails decoding notification missing subject`() { + @Test func failsDecodingNotificationMissingSubject() { let json = """ { "id": 7, @@ -182,7 +182,7 @@ struct NotificationTests { } } - @Test func `fails decoding notification missing repository`() { + @Test func failsDecodingNotificationMissingRepository() { let json = """ { "id": 8, @@ -197,7 +197,7 @@ struct NotificationTests { } } - @Test func `fails decoding notification missing id`() { + @Test func failsDecodingNotificationMissingId() { let json = """ { "unread": true, @@ -212,14 +212,14 @@ struct NotificationTests { } } - @Test func `fails decoding notification count missing new field`() { + @Test func failsDecodingNotificationCountMissingNewField() { let json = #"{}"# #expect(throws: DecodingError.self) { _ = try decoder().decode(NotificationCount.self, from: Data(json.utf8)) } } - @Test func `fails decoding subject missing title`() { + @Test func failsDecodingSubjectMissingTitle() { let json = """ { "id": 9, diff --git a/Tests/ForgejoKitTests/RepositoryServiceURLTests.swift b/Tests/ForgejoKitTests/RepositoryServiceURLTests.swift index ba51610..ad0ca84 100644 --- a/Tests/ForgejoKitTests/RepositoryServiceURLTests.swift +++ b/Tests/ForgejoKitTests/RepositoryServiceURLTests.swift @@ -10,20 +10,20 @@ struct RepositoryServiceURLTests { // MARK: - fetchContents URL construction - @Test func `make URL contents without ref`() throws { + @Test func makeURLContentsWithoutRef() throws { let client = makeClient() let url = try client.makeURL(path: "/api/v1/repos/owner/repo/contents") #expect(url.absoluteString == "https://forgejo.example.com/api/v1/repos/owner/repo/contents") } - @Test func `make URL contents with ref`() throws { + @Test func makeURLContentsWithRef() throws { let client = makeClient() let queryItems = [URLQueryItem(name: "ref", value: "develop")] let url = try client.makeURL(path: "/api/v1/repos/owner/repo/contents", queryItems: queryItems) #expect(url.absoluteString == "https://forgejo.example.com/api/v1/repos/owner/repo/contents?ref=develop") } - @Test func `make URL contents with ref and subpath`() throws { + @Test func makeURLContentsWithRefAndSubpath() throws { let client = makeClient() let queryItems = [URLQueryItem(name: "ref", value: "feature/login")] let url = try client.makeURL(path: "/api/v1/repos/owner/repo/contents/src/main.py", queryItems: queryItems) @@ -31,14 +31,14 @@ struct RepositoryServiceURLTests { #expect(url.path == "/api/v1/repos/owner/repo/contents/src/main.py") } - @Test func `make URL file content with ref`() throws { + @Test func makeURLFileContentWithRef() throws { let client = makeClient() let queryItems = [URLQueryItem(name: "ref", value: "v1.0")] let url = try client.makeURL(path: "/api/v1/repos/owner/repo/contents/README.md", queryItems: queryItems) #expect(url.absoluteString == "https://forgejo.example.com/api/v1/repos/owner/repo/contents/README.md?ref=v1.0") } - @Test func `make URL with empty query items`() throws { + @Test func makeURLWithEmptyQueryItems() throws { let client = makeClient() let url = try client.makeURL(path: "/api/v1/repos/owner/repo/contents", queryItems: []) #expect(url.query == nil) @@ -46,13 +46,13 @@ struct RepositoryServiceURLTests { // MARK: - ref parameter nil-coalescence (same pattern used by fetchContents and fetchFileContent) - @Test func `optional ref produces empty query items`() { + @Test func optionalRefProducesEmptyQueryItems() { let ref: String? = nil let queryItems = ref.map { [URLQueryItem(name: "ref", value: $0)] } ?? [] #expect(queryItems.isEmpty) } - @Test func `present ref produces query item`() { + @Test func presentRefProducesQueryItem() { let ref: String? = "main" let queryItems = ref.map { [URLQueryItem(name: "ref", value: $0)] } ?? [] #expect(queryItems.count == 1) @@ -62,7 +62,7 @@ struct RepositoryServiceURLTests { // MARK: - createRepository URL - @Test func `create repository URL`() throws { + @Test func createRepositoryURL() throws { let client = makeClient() let url = try client.makeURL(path: "/api/v1/user/repos") #expect(url.absoluteString == "https://forgejo.example.com/api/v1/user/repos") @@ -70,7 +70,7 @@ struct RepositoryServiceURLTests { // MARK: - editRepository URL - @Test func `edit repository URL`() throws { + @Test func editRepositoryURL() throws { let client = makeClient() let url = try client.makeRepoURL(owner: "owner", repo: "repo", path: "") #expect(url.absoluteString == "https://forgejo.example.com/api/v1/repos/owner/repo") @@ -78,7 +78,7 @@ struct RepositoryServiceURLTests { // MARK: - createLabel URL - @Test func `create label URL`() throws { + @Test func createLabelURL() throws { let client = makeClient() let url = try client.makeRepoURL(owner: "owner", repo: "repo", path: "/labels") #expect(url.absoluteString == "https://forgejo.example.com/api/v1/repos/owner/repo/labels") @@ -86,7 +86,7 @@ struct RepositoryServiceURLTests { // MARK: - createMilestone URL - @Test func `create milestone URL`() throws { + @Test func createMilestoneURL() throws { let client = makeClient() let url = try client.makeRepoURL(owner: "owner", repo: "repo", path: "/milestones") #expect(url.absoluteString == "https://forgejo.example.com/api/v1/repos/owner/repo/milestones") @@ -94,7 +94,7 @@ struct RepositoryServiceURLTests { // MARK: - addCollaborator URL - @Test func `add collaborator URL`() throws { + @Test func addCollaboratorURL() throws { let client = makeClient() let encoded = ForgejoClient.encodedPathSegment("testbot") let url = try client.makeRepoURL(owner: "owner", repo: "repo", path: "/collaborators/\(encoded)") @@ -103,7 +103,7 @@ struct RepositoryServiceURLTests { // MARK: - createFile URL - @Test func `create file URL`() throws { + @Test func createFileURL() throws { let client = makeClient() let url = try client.makeRepoURL(owner: "owner", repo: "repo", path: "/contents/hello.py") #expect(url.absoluteString == "https://forgejo.example.com/api/v1/repos/owner/repo/contents/hello.py") @@ -111,7 +111,7 @@ struct RepositoryServiceURLTests { // MARK: - Payload encoding - @Test func `create repository payload encoding`() throws { + @Test func createRepositoryPayloadEncoding() throws { let payload: [String: Any] = [ "name": "test-repo", "description": "A test repo", @@ -125,7 +125,7 @@ struct RepositoryServiceURLTests { #expect(dict["private"] as? Bool == false) } - @Test func `create file payload encoding`() throws { + @Test func createFilePayloadEncoding() throws { let payload: [String: Any] = [ "content": "base64content", "message": "Add file", diff --git a/Tests/ForgejoKitTests/URLSessionManagerTests.swift b/Tests/ForgejoKitTests/URLSessionManagerTests.swift index c151384..9106d53 100644 --- a/Tests/ForgejoKitTests/URLSessionManagerTests.swift +++ b/Tests/ForgejoKitTests/URLSessionManagerTests.swift @@ -5,22 +5,22 @@ import Testing struct URLSessionManagerTests { // MARK: - trustedHost scoping - @Test func `self signed without trusted host accepts any`() { + @Test func selfSignedWithoutTrustedHostAcceptsAny() { let manager = URLSessionManager(allowSelfSignedCertificates: true, trustedHost: nil) #expect(manager.allowSelfSignedCertificates == true) } - @Test func `self signed with trusted host is set`() { + @Test func selfSignedWithTrustedHostIsSet() { let manager = URLSessionManager(allowSelfSignedCertificates: true, trustedHost: "forgejo.example.com") #expect(manager.allowSelfSignedCertificates == true) } - @Test func `self signed disabled by default`() { + @Test func selfSignedDisabledByDefault() { let manager = URLSessionManager() #expect(manager.allowSelfSignedCertificates == false) } - @Test func `session is created with configuration`() { + @Test func sessionIsCreatedWithConfiguration() { let manager = URLSessionManager(allowSelfSignedCertificates: false) let session = manager.session #expect(session.configuration.timeoutIntervalForRequest == 30) @@ -29,31 +29,31 @@ struct URLSessionManagerTests { // MARK: - Trust disposition logic - @Test func `self signed disabled returns default handling`() { + @Test func selfSignedDisabledReturnsDefaultHandling() { let manager = URLSessionManager(allowSelfSignedCertificates: false) let disposition = manager.trustDisposition(for: "forgejo.example.com") #expect(disposition == .performDefaultHandling) } - @Test func `self signed enabled with nil trusted host returns default handling`() { + @Test func selfSignedEnabledWithNilTrustedHostReturnsDefaultHandling() { let manager = URLSessionManager(allowSelfSignedCertificates: true, trustedHost: nil) let disposition = manager.trustDisposition(for: "forgejo.example.com") #expect(disposition == .performDefaultHandling) } - @Test func `self signed enabled with matching host returns use credential`() { + @Test func selfSignedEnabledWithMatchingHostReturnsUseCredential() { let manager = URLSessionManager(allowSelfSignedCertificates: true, trustedHost: "forgejo.example.com") let disposition = manager.trustDisposition(for: "forgejo.example.com") #expect(disposition == .useCredential) } - @Test func `self signed enabled with mismatched host returns default handling`() { + @Test func selfSignedEnabledWithMismatchedHostReturnsDefaultHandling() { let manager = URLSessionManager(allowSelfSignedCertificates: true, trustedHost: "forgejo.example.com") let disposition = manager.trustDisposition(for: "evil.example.com") #expect(disposition == .performDefaultHandling) } - @Test func `self signed trust matches case insensitive`() { + @Test func selfSignedTrustMatchesCaseInsensitive() { let manager = URLSessionManager(allowSelfSignedCertificates: true, trustedHost: "Forgejo.Example.COM") let disposition = manager.trustDisposition(for: "forgejo.example.com") #expect(disposition == .useCredential) @@ -63,7 +63,7 @@ struct URLSessionManagerTests { struct ForgejoClientHostMatchTests { // MARK: - Auth header host matching - @Test func `authenticated request includes auth for matching host`() throws { + @Test func authenticatedRequestIncludesAuthForMatchingHost() throws { let client = ForgejoClient( serverURL: "https://forgejo.example.com", username: "user", @@ -74,7 +74,7 @@ struct ForgejoClientHostMatchTests { #expect(request.value(forHTTPHeaderField: "Authorization") != nil) } - @Test func `authenticated request omits auth for different host`() throws { + @Test func authenticatedRequestOmitsAuthForDifferentHost() throws { let client = ForgejoClient( serverURL: "https://forgejo.example.com", username: "user", @@ -85,7 +85,7 @@ struct ForgejoClientHostMatchTests { #expect(request.value(forHTTPHeaderField: "Authorization") == nil) } - @Test func `authenticated request matches case insensitive host`() throws { + @Test func authenticatedRequestMatchesCaseInsensitiveHost() throws { let client = ForgejoClient( serverURL: "https://Forgejo.Example.COM", username: "user", @@ -96,7 +96,7 @@ struct ForgejoClientHostMatchTests { #expect(request.value(forHTTPHeaderField: "Authorization") != nil) } - @Test func `authenticated request sets method and body`() throws { + @Test func authenticatedRequestSetsMethodAndBody() throws { let client = ForgejoClient( serverURL: "https://forgejo.example.com", username: "user", @@ -110,7 +110,7 @@ struct ForgejoClientHostMatchTests { #expect(request.value(forHTTPHeaderField: "Content-Type") == "application/json") } - @Test func `make URL constructs valid URL`() throws { + @Test func makeURLConstructsValidURL() throws { let client = ForgejoClient( serverURL: "https://forgejo.example.com", username: "user", @@ -120,7 +120,7 @@ struct ForgejoClientHostMatchTests { #expect(url.absoluteString == "https://forgejo.example.com/api/v1/user") } - @Test func `make URL with query items`() throws { + @Test func makeURLWithQueryItems() throws { let client = ForgejoClient( serverURL: "https://forgejo.example.com", username: "user", @@ -134,7 +134,7 @@ struct ForgejoClientHostMatchTests { #expect(url.absoluteString.contains("page=2")) } - @Test func `client extracts host for trusted cert`() { + @Test func clientExtractsHostForTrustedCert() { // Verify that the client passes the right host to URLSessionManager let client = ForgejoClient( serverURL: "https://my-forgejo.local:3000", diff --git a/Tests/ForgejoKitTests/UserServiceTests.swift b/Tests/ForgejoKitTests/UserServiceTests.swift index 518706a..74c340a 100644 --- a/Tests/ForgejoKitTests/UserServiceTests.swift +++ b/Tests/ForgejoKitTests/UserServiceTests.swift @@ -3,7 +3,7 @@ import Foundation import Testing struct UserServiceTests { - @Test func `create token payload encodes correct keys`() throws { + @Test func createTokenPayloadEncodesCorrectKeys() throws { let payload: [String: Any] = [ "name": "integration-test-token", "scopes": ["read:user", "write:user"], @@ -14,13 +14,13 @@ struct UserServiceTests { #expect((dict["scopes"] as? [String])?.count == 2) } - @Test func `create token URL construction`() throws { + @Test func createTokenURLConstruction() throws { let client = ForgejoClient(serverURL: "https://forgejo.example.com", username: "admin", password: "pass") let url = try client.makeURL(path: "/api/v1/users/\(ForgejoClient.encodedPathSegment("testuser"))/tokens") #expect(url.absoluteString == "https://forgejo.example.com/api/v1/users/testuser/tokens") } - @Test func `create token URL encodes special characters`() throws { + @Test func createTokenURLEncodesSpecialCharacters() throws { let client = ForgejoClient(serverURL: "https://forgejo.example.com", username: "admin", password: "pass") let encoded = ForgejoClient.encodedPathSegment("user name") let url = try client.makeURL(path: "/api/v1/users/\(encoded)/tokens") diff --git a/Tests/ForgejoKitTests/WorkflowServiceURLTests.swift b/Tests/ForgejoKitTests/WorkflowServiceURLTests.swift index 208f03d..5b3c859 100644 --- a/Tests/ForgejoKitTests/WorkflowServiceURLTests.swift +++ b/Tests/ForgejoKitTests/WorkflowServiceURLTests.swift @@ -7,7 +7,7 @@ struct WorkflowServiceURLTests { ForgejoClient(serverURL: "https://forgejo.example.com", username: "user", password: "pass") } - @Test func `runs list URL no filters`() throws { + @Test func runsListURLNoFilters() throws { let client = makeClient() let queryItems = [ URLQueryItem(name: "page", value: "1"), @@ -22,7 +22,7 @@ struct WorkflowServiceURLTests { #expect(url.query?.contains("limit=20") == true) } - @Test func `runs list URL all filters`() throws { + @Test func runsListURLAllFilters() throws { let client = makeClient() let queryItems = [ URLQueryItem(name: "page", value: "2"), @@ -46,13 +46,13 @@ struct WorkflowServiceURLTests { #expect(url.query?.contains("head_sha=abc123") == true) } - @Test func `run detail URL`() throws { + @Test func runDetailURL() throws { let client = makeClient() let url = try client.makeRepoURL(owner: "owner", repo: "repo", path: "/actions/runs/123") #expect(url.absoluteString == "https://forgejo.example.com/api/v1/repos/owner/repo/actions/runs/123") } - @Test func `owner and repo are percent encoded`() throws { + @Test func ownerAndRepoArePercentEncoded() throws { let client = makeClient() let url = try client.makeRepoURL(owner: "my org", repo: "my repo", path: "/actions/runs") #expect(url.absoluteString.contains("my%20org")) @@ -61,13 +61,13 @@ struct WorkflowServiceURLTests { // MARK: - Experimental run-view URL (web routes, not /api/v1) - @Test func `run view URL is under repo web route`() throws { + @Test func runViewURLIsUnderRepoWebRoute() throws { let client = makeClient() let url = try client.makeURL(path: "/owner/repo/actions/runs/12/jobs/0") #expect(url.absoluteString == "https://forgejo.example.com/owner/repo/actions/runs/12/jobs/0") } - @Test func `run view URL encodes owner and repo`() throws { + @Test func runViewURLEncodesOwnerAndRepo() throws { let client = makeClient() let owner = ForgejoClient.encodedPathSegment("my org") let repo = ForgejoClient.encodedPathSegment("my repo")