Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fehler #2

Open
wants to merge 16 commits into
base: master
Choose a base branch
from
Open

Fehler #2

wants to merge 16 commits into from

Conversation

DenKlei
Copy link

@DenKlei DenKlei commented May 24, 2016

Ich bekomme im PlanetViewController das der initializer fehlt. Ich habe es zwar mit dem pokedex verglichen, aber ich hab nichts dergleichen gefunden, was mir weitergeholfen hätte. Habe es auch noch mit required init probiert, aber damit komme ich nicht viel weiter außer dass es kompiliert.


enum PlanetAPI: Moya.TargetType{
case planet(name: String)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Du kannst hier auch eine NamedResource<Planet> statt eines name: String als Associated Value verwenden, damit wird die API noch besser modelliert ;)

@DenKlei
Copy link
Author

DenKlei commented May 25, 2016

Das habe ich nun umgesetzt, leider bekomme ich nun im AppDelegate einen Fehler mit dem ich nichts anfangen kann. Würde eigentlich nur mal wissen wollen ob ich einen response erhalte :/

@nilsvu
Copy link
Member

nilsvu commented May 26, 2016

Hey, du hast nur zwei ungültige Outlets im Storyboard:
bildschirmfoto 2016-05-26 um 02 36 08

Damit textFieldShouldReturn musst du auch noch das delegate des Textfelds auf den View Controller setzen.

@DenKlei
Copy link
Author

DenKlei commented May 27, 2016

Ich bekomme jetzt eine Meldung, dass meine Constraints nicht stimmen. Ich schätze mal, dass ist auch der Grund wieso alles schwarz unter der Suchleiste ist. Was läuft da schief?

@nilsvu
Copy link
Member

nilsvu commented May 27, 2016

Hey, zieh' eine UIView auf den View Controller (das ist dann dessen Content View) und positioniere die Stack View dann darin:
bildschirmfoto 2016-05-27 um 14 01 15
Die Stack View kannst du dann mit Constraints an den Rand der Content View kleben.

var path: String {
switch self{
case .planet(let namedResource): return "/planets/(namedResource)"
}
Copy link
Member

@nilsvu nilsvu May 27, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hier fehlt ein \, so ist das nur ein String, und kein Wert wird eingesetzt ;) Daher kommt momentan immer der Status Code 404 zurück.

        case .planet(let namedResource): return "/planets/\(namedResource.name)"

@DenKlei
Copy link
Author

DenKlei commented May 30, 2016

Danke :)

@DenKlei
Copy link
Author

DenKlei commented May 30, 2016

Erstmal sry für die vielen commits aber irgendwie gab es ein Problem mit der online Version.
Wie sehe ich die Coverage? Ein Test ist erfolgreich, aber ich kann nicht die Coverage sehen im Report Navigator.

Im PlanetAPIUITests findet er das Textfeld nicht, obwohl es genau gleich heißt. Hab auch den Identifier gesetzt

@nilsvu
Copy link
Member

nilsvu commented May 30, 2016

Sorry hab vergessen, dass nach dem pull --rebase die Commitfolge verändert ist und ein einfacher push nicht mehr klappt. Gut, dass du's hinbekommen hast. Hab's im App Katalog nachgetragen.

@nilsvu
Copy link
Member

nilsvu commented May 30, 2016

  • Das Textfeld hat im Storyboard nicht den richtigen Identifier, so sollte es aussehen:
    bildschirmfoto 2016-05-30 um 19 44 24
  • Die Coverage sollte hier angezeigt werden:
    bildschirmfoto 2016-05-30 um 19 46 39
    Wenn nicht, drück CMD+< und wähl es hier aus:
    bildschirmfoto 2016-05-30 um 19 47 50

@DenKlei
Copy link
Author

DenKlei commented May 31, 2016

Ich habe jetzt noch ein ganz anderes Problem. Gestern lief mein PlanetAPITests noch einwandfrei und jetzt habe ich einen Fehler im main.storyboard und zwar Line 47: StartTag: invalid element name. Jetzt kann ich nicht mal mehr auf das storyboard zu greifen, mir zeigt es nur den Fehler an

</subviews>
<color key="backgroundColor" red="1" green="0.0" blue="0.0" alpha="1" colorSpace="calibratedRGB"/>
<<<<<<< HEAD
<accessibility key="accessibilityConfiguration" identifier="accessibilityIdentifier">
Copy link
Member

@nilsvu nilsvu Jun 2, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dies sind Konflikt-Marker von Git nach einem Merge. Öffne das Storyboard in einem Texteditor und entferne die Marker und damit die Konflikte ;) https://githowto.com/resolving_conflicts

Kleimaier and others added 2 commits June 2, 2016 18:36
@DenKlei
Copy link
Author

DenKlei commented Jun 2, 2016

Wie kann man folgendes schreiben: expect(searchTextfield.label).notTo(beEmpty())
Weil jetzt sagt er mir das Textfeld ist nicht empty, aber das steht ja auch da.
Und bei app.staticTexts meckert er iwie, aber das Label ist als StaticText markiert.

@nilsvu
Copy link
Member

nilsvu commented Jun 5, 2016

API-Abfrage funktioniert 👍 2/2 Pt. Für mehr Funktionalität gibt's Extrapunkte.

@nilsvu
Copy link
Member

nilsvu commented Jun 5, 2016

  • Dein Textfeld ist zu Beginn ja nicht leer, das hat dein Test richtig erkannt! Anstatt das im Test zu umgehen frage dich lieber: Es sollte aber leer sein, dein Test hat einen Bug erkannt, wie kannst du den beheben? Entfernen also den Anfangs-Text im Storyboard und setze ihn lieber als Placeholder!
  • Setz mal diese waitForResponsiveness Workaround-Methode an den Anfang, die ich in meinem Beispiel benutze. UI Tests sind noch nicht so komplett ausgereift ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants