Skip to content

test: add focused coverage for DateValue and MQTT model paths#12

Draft
augard wants to merge 1 commit intomainfrom
chore/test-gap-mqtt-datevalue-20260307
Draft

test: add focused coverage for DateValue and MQTT model paths#12
augard wants to merge 1 commit intomainfrom
chore/test-gap-mqtt-datevalue-20260307

Conversation

@augard
Copy link
Owner

@augard augard commented Mar 9, 2026

Summary

  • replace placeholder KiaTests.swift examples with focused unit/regression tests for recent PR ✨ Integrate MQTT real-time updates and improve authentication #6 model changes
  • add coverage for DateValue decode/encode paths in DatePropertyWrapper.swift
  • add coverage for MQTT protocol topic mapping and request encoding in MQTTModels.swift

Tests added

  • timestamp milliseconds string decoding
  • native Date decoding path via decoder strategy
  • invalid timestamp parse failure path
  • millisecond date formatter encode/decode round-trip
  • MQTT subscriptionName mapping
  • MQTT init(topicName:) success and unknown topic rejection
  • ProtocolSubscriptionRequest raw-value JSON encoding
  • MQTTConnectionStatus.displayText mapping

Verification

  • tests introduced in KiaTests/KiaTests.swift
  • local simulator-based xcodebuild test was not run in the previous sandboxed environment

@augard augard added codex Created by codex automation codex-automation Created by codex automation labels Mar 9, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Mar 9, 2026

There is a problem with the Gemini CLI PR review. Please check the action logs for details.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

codex Created by codex automation codex-automation Created by codex automation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant