Packages/Pachyderm/Sources/Pachyderm/Model/Status.swift

@@ -51,12 +51,8 @@ public final class Status: StatusProtocol, Decodable, Sendable {
         do {
             self.url = try container.decodeIfPresent(WebURL.self, forKey: .url)
         } catch {
-            let s = try? container.decode(String.self, forKey: .url)
+            let s = (try? container.decode(String.self, forKey: .url)) ?? "<failed to decode string>"
-            if s == "" {
+            throw DecodingError.dataCorruptedError(forKey: .url, in: container, debugDescription: "Could not decode URL '\(s)', reason: \(String(describing: error))")
-                self.url = nil
-            } else {
-                throw DecodingError.dataCorrupted(.init(codingPath: container.codingPath + [CodingKeys.url], debugDescription: "Could not decode URL '\(s ?? "<failed to decode string>")'", underlyingError: error))
-            }
         }
         self.account = try container.decode(Account.self, forKey: .account)
         self.inReplyToID = try container.decodeIfPresent(String.self, forKey: .inReplyToID)

Packages/Pachyderm/Tests/PachydermTests/CharacterCounterTests.swift

@@ -1,14 +1,13 @@
 //
 //  CharacterCounterTests.swift
-//  ComposeUITests
+//  PachydermTests
 //
 //  Created by Shadowfacts on 9/29/18.
 //  Copyright © 2018 Shadowfacts. All rights reserved.
 //
 
 import XCTest
-@testable import ComposeUI
+@testable import Pachyderm
-import InstanceFeatures
 
 class CharacterCounterTests: XCTestCase {
 
@@ -18,33 +17,31 @@ class CharacterCounterTests: XCTestCase {
     override func tearDown() {
     }
 
-    let features = InstanceFeatures()
-
     func testCountEmpty() {
-        XCTAssertEqual(CharacterCounter.count(text: "", for: features), 0)
+        XCTAssertEqual(CharacterCounter.count(text: ""), 0)
     }
     
     func testCountPlainText() {
-        XCTAssertEqual(CharacterCounter.count(text: "This is an example message", for: features), 26)
+        XCTAssertEqual(CharacterCounter.count(text: "This is an example message"), 26)
-        XCTAssertEqual(CharacterCounter.count(text: "This is an example message with an Emoji: 😄", for: features), 43)
+        XCTAssertEqual(CharacterCounter.count(text: "This is an example message with an Emoji: 😄"), 43)
-        XCTAssertEqual(CharacterCounter.count(text: "😄😄😄😄😄😄😄", for: features), 7)
+        XCTAssertEqual(CharacterCounter.count(text: "😄😄😄😄😄😄😄"), 7)
     }
     
     func testCountLinks() {
-        XCTAssertEqual(CharacterCounter.count(text: "This is an example with a link: https://example.com", for: features), 55)
+        XCTAssertEqual(CharacterCounter.count(text: "This is an example with a link: https://example.com"), 55)
-        XCTAssertEqual(CharacterCounter.count(text: "This is an example with a link 😄: https://example.com", for: features), 57)
+        XCTAssertEqual(CharacterCounter.count(text: "This is an example with a link 😄: https://example.com"), 57)
-        XCTAssertEqual(CharacterCounter.count(text: "😄😄😄😄😄😄😄: https://example.com", for: features), 32)
+        XCTAssertEqual(CharacterCounter.count(text: "😄😄😄😄😄😄😄: https://example.com"), 32)
-        XCTAssertEqual(CharacterCounter.count(text: "This is an example with a link: https://a.much.longer.example.com/link?foo=bar#baz", for: features), 55)
+        XCTAssertEqual(CharacterCounter.count(text: "This is an example with a link: https://a.much.longer.example.com/link?foo=bar#baz"), 55)
     }
     
     func testCountLocalMentions() {
-        XCTAssertEqual(CharacterCounter.count(text: "hello @example", for: features), 14)
+        XCTAssertEqual(CharacterCounter.count(text: "hello @example"), 14)
-        XCTAssertEqual(CharacterCounter.count(text: "@some_really_long_name", for: features), 22)
+        XCTAssertEqual(CharacterCounter.count(text: "@some_really_long_name"), 22)
     }
     
     func testCountRemoteMentions() {
-        XCTAssertEqual(CharacterCounter.count(text: "hello @example@some.remote.social", for: features), 14)
+        XCTAssertEqual(CharacterCounter.count(text: "hello @example@some.remote.social"), 14)
-        XCTAssertEqual(CharacterCounter.count(text: "hello @some_really_long_name@some-long.remote-instance.social", for: features), 28)
+        XCTAssertEqual(CharacterCounter.count(text: "hello @some_really_long_name@some-long.remote-instance.social"), 28)
     }
 
 }

Packages/Pachyderm/Tests/PachydermTests/NotificationGroupTests.swift

@@ -8,7 +8,6 @@
 import XCTest
 @testable import Pachyderm
 
-@MainActor
 class NotificationGroupTests: XCTestCase {
     
     let decoder: JSONDecoder = {

Packages/Pachyderm/Tests/PachydermTests/StatusTests.swift

@@ -1,53 +0,0 @@
-//
-//  StatusTests.swift
-//  
-//
-//  Created by Shadowfacts on 5/8/23.
-//
-
-import XCTest
-@testable import Pachyderm
-
-final class StatusTests: XCTestCase {
-
-    func testDecode() {
-        let data = """
-{
-    "id": "1",
-    "uri": "https://example.com/a/1",
-    "url": "",
-    "account": {
-        "id": "2",
-        "username": "a",
-        "acct": "a",
-        "display_name": "",
-        "locked": false,
-        "created_at": 0,
-        "followers_count": 0,
-        "following_count": 0,
-        "statuses_count": 0,
-        "note": "",
-        "url": "https://example.com/a"
-    },
-    "content": "",
-    "created_at": 0,
-    "emojis": [],
-    "reblogs_count": 0,
-    "favourites_count":  0,
-    "sensitive": false,
-    "spoiler_text": "",
-    "visibility": "public",
-    "media_attachments": [],
-    "mentions": [],
-    "tags": []
-}
-""".data(using: .utf8)!
-        do {
-            _ = try JSONDecoder().decode(Status.self, from: data)
-        } catch {
-            print(error)
-            XCTFail()
-        }
-    }
-    
-}

Tusker.xcodeproj/xcshareddata/xcschemes/Pachyderm.xcscheme

@@ -17,7 +17,7 @@
                BlueprintIdentifier = "D61099AA2144B0CC00432DC2"
                BuildableName = "Pachyderm.framework"
                BlueprintName = "Pachyderm"
-               ReferencedContainer = "container:../../Tusker.xcodeproj">
+               ReferencedContainer = "container:Tusker.xcodeproj">
             </BuildableReference>
          </BuildActionEntry>
       </BuildActionEntries>
@@ -35,7 +35,7 @@
                BlueprintIdentifier = "PachydermTests"
                BuildableName = "PachydermTests"
                BlueprintName = "PachydermTests"
-               ReferencedContainer = "container:">
+               ReferencedContainer = "container:Pachyderm">
             </BuildableReference>
          </TestableReference>
       </Testables>
@@ -56,7 +56,7 @@
             BlueprintIdentifier = "D61099AA2144B0CC00432DC2"
             BuildableName = "Pachyderm.framework"
             BlueprintName = "Pachyderm"
-            ReferencedContainer = "container:../../Tusker.xcodeproj">
+            ReferencedContainer = "container:Tusker.xcodeproj">
          </BuildableReference>
       </MacroExpansion>
    </LaunchAction>
@@ -72,7 +72,7 @@
             BlueprintIdentifier = "D61099AA2144B0CC00432DC2"
             BuildableName = "Pachyderm.framework"
             BlueprintName = "Pachyderm"
-            ReferencedContainer = "container:../../Tusker.xcodeproj">
+            ReferencedContainer = "container:Tusker.xcodeproj">
          </BuildableReference>
       </MacroExpansion>
    </ProfileAction>

Tusker/Screens/Notifications/NotificationsCollectionViewController.swift

@@ -355,24 +355,8 @@ extension NotificationsCollectionViewController {
     }
     
     func handlePrependItems(_ timelineItems: [NotificationGroup]) async {
-        let topItem = dataSource.snapshot().itemIdentifiers(inSection: .notifications).first
-        
         // we always replace all, because new items are merged with existing ones
         await handleReplaceAllItems(timelineItems)
-        
-        // preserve the scroll position
-        // todo: this won't work for cmd+r when not at top
-        if let topID = topItem?.group?.notifications.first?.id {
-            // the exact item may have changed, due to merging
-            let newTopGroup = timelineItems.first {
-                $0.notifications.contains {
-                    $0.id == topID
-                }
-            }!
-            if let newTopIndexPath = dataSource.indexPath(for: .group(newTopGroup)) {
-                collectionView.scrollToItem(at: newTopIndexPath, at: .top, animated: false)
-            }
-        }
     }
     
     func loadOlder() async throws -> [NotificationGroup] {

Tusker/Screens/Utilities/TimelineLikeCollectionViewController.swift

@@ -153,7 +153,6 @@ extension TimelineLikeCollectionViewController {
         }
         await apply(snapshot, animatingDifferences: false)
         
-        // todo: this won't work for cmd+r when not at top
         if let first,
            let indexPath = dataSource.indexPath(for: first) {
             // TODO: i can't tell if this actually works or not