Mrixs/trip-planner
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

feat: implement Yandex /search method for on-demand edge expansion (Task 2)

Browse Source 
- Add SearchRoutes method to yandex client for on-demand station pair searches
- Implement hub expansion via on-demand /search calls in lazy graph expansion
- Integrate cache key generation for search results (search:{from}:{to}:{date})
- Update expandFromStation and expandFromCityHub to use Yandex API
- Add NewGraphWithoutYandex constructor for testability
- Add tests for on-demand search integration and cache TTL policies
This commit is contained in:
Vladimir Zagainov
2026-08-14 14:14:14 +03:00
parent 829e93fc8f
commit 181575e092
5 changed files with 337 additions and 84 deletions
Download Patch File Download Diff File Expand all files Collapse all files

103
internal/routing/graph_test.go
View File

@@ -1,7 +1,11 @@
package routing
import (
"context"
"testing"
"trip-planner/internal/cache"
"trip-planner/internal/yandex"
)
func TestGraphNodeCreation(t *testing.T) {
@@ -79,7 +83,7 @@ func TestGraphEdgeCreation(t *testing.T) {
}
func TestGraphAddNodeAndEdge(t *testing.T) {
graph := NewGraph()
graph := NewGraphWithoutYandex()
node := &Node{ID: "n1", Type: NodeTypeStation, Name: "Test Station"}
graph.AddNode(node)
@@ -160,7 +164,7 @@ func TestBuildGraphFromStations(t *testing.T) {
}
func TestGraphNodesAndEdges(t *testing.T) {
graph := NewGraph()
graph := NewGraphWithoutYandex()
// Add nodes
graph.AddNode(&Node{ID: "n1", Type: NodeTypeStation, Name: "Station 1"})
@@ -181,7 +185,7 @@ func TestGraphNodesAndEdges(t *testing.T) {
}
func TestFindRouteSuccess(t *testing.T) {
graph := NewGraph()
graph := NewGraphWithoutYandex()
// Add stations
graph.AddNode(&Node{ID: "s1", Type: NodeTypeStation, Name: "Moscow", CityCode: "c1"})
@@ -210,7 +214,7 @@ func TestFindRouteSuccess(t *testing.T) {
}
func TestFindRouteNoRoute(t *testing.T) {
graph := NewGraph()
graph := NewGraphWithoutYandex()
// Add isolated nodes with no connections
graph.AddNode(&Node{ID: "s1", Type: NodeTypeStation, Name: "Station 1", CityCode: "c1"})
@@ -226,7 +230,7 @@ func TestFindRouteNoRoute(t *testing.T) {
}
func TestFindRouteExceedsTransferLimit(t *testing.T) {
graph := NewGraph()
graph := NewGraphWithoutYandex()
// Add a chain of stations with synthetic transfer edges (would require 4 transfers)
graph.AddNode(&Node{ID: "s1", Type: NodeTypeStation, Name: "Moscow", CityCode: "c1"})
@@ -251,7 +255,7 @@ func TestFindRouteExceedsTransferLimit(t *testing.T) {
}
func TestApplyMCT_CityHubReducesMCT(t *testing.T) {
graph := NewGraph()
graph := NewGraphWithoutYandex()
// Create legs with city hub transfers
graph.AddNode(&Node{ID: "s1", Type: NodeTypeStation, Name: "Moscow", CityCode: "c1"})
@@ -281,7 +285,7 @@ func TestApplyMCT_CityHubReducesMCT(t *testing.T) {
}
func TestApplyMCT_ModeChangeIncreasesMCT(t *testing.T) {
graph := NewGraph()
graph := NewGraphWithoutYandex()
// Create legs with mode change
graph.AddNode(&Node{ID: "s1", Type: NodeTypeStation, Name: "Moscow", CityCode: "c1"})
@@ -308,7 +312,7 @@ func TestApplyMCT_ModeChangeIncreasesMCT(t *testing.T) {
}
func TestApplyMCT_ModeChangeBetweenLegs(t *testing.T) {
graph := NewGraph()
graph := NewGraphWithoutYandex()
// Create 2 stations for 2 legs with mode change (train then bus)
graph.AddNode(&Node{ID: "s1", Type: NodeTypeStation, Name: "Moscow", CityCode: "c1"})
@@ -339,7 +343,7 @@ func TestApplyMCT_ModeChangeBetweenLegs(t *testing.T) {
// TestFindRoutesPareto tests the Pareto-optimal route finding.
func TestFindRoutesPareto(t *testing.T) {
graph := NewGraph()
graph := NewGraphWithoutYandex()
// Add stations along a route
graph.AddNode(&Node{ID: "s1", Type: NodeTypeStation, Name: "Moscow", CityCode: "c1"})
@@ -406,7 +410,7 @@ func TestFindRoutesPareto(t *testing.T) {
// TestFindRouteWith2Transfers tests route finding with exactly 2 transfers.
func TestFindRouteWith2Transfers(t *testing.T) {
graph := NewGraph()
graph := NewGraphWithoutYandex()
// Add stations: A -> B -> C -> D (3 hops, 2 transfers)
graph.AddNode(&Node{ID: "a", Type: NodeTypeStation, Name: "A", CityCode: "c1"})
@@ -433,7 +437,7 @@ func TestFindRouteWith2Transfers(t *testing.T) {
// TestFindRouteExactly2Transfers tests route with exactly 2 transfers is rejected at 1.
func TestFindRouteExactly2TransfersRejectedAt1(t *testing.T) {
graph := NewGraph()
graph := NewGraphWithoutYandex()
graph.AddNode(&Node{ID: "s1", Type: NodeTypeStation, Name: "Moscow", CityCode: "c1"})
graph.AddNode(&Node{ID: "s2", Type: NodeTypeStation, Name: "Tula", CityCode: "c1"})
@@ -460,7 +464,7 @@ func TestFindRouteExactly2TransfersRejectedAt1(t *testing.T) {
// TestApplyMCT_MultipleTransfers tests MCT application with multiple transfers.
func TestApplyMCT_MultipleTransfers(t *testing.T) {
graph := NewGraph()
graph := NewGraphWithoutYandex()
graph.AddNode(&Node{ID: "s1", Type: NodeTypeStation, Name: "Moscow", CityCode: "c1"})
graph.AddNode(&Node{ID: "s2", Type: NodeTypeCity, Name: "City1", CityCode: "c1"})
@@ -662,7 +666,7 @@ func TestBuildGraphFromHubs(t *testing.T) {
// TestExpandGraphLazy tests the lazy graph expansion method.
func TestExpandGraphLazy(t *testing.T) {
graph := NewGraph()
graph := NewGraphWithoutYandex()
// Add a station node
moscow := &Node{
@@ -673,7 +677,7 @@ func TestExpandGraphLazy(t *testing.T) {
graph.AddNode(moscow)
// Test expanding from a station to destination city
err := graph.ExpandGraphLazy(moscow, "Simferopol")
err := graph.ExpandGraphLazy(moscow, "Simferopol", "2026-08-20")
if err != nil {
t.Errorf("expected no error from ExpandGraphLazy, got: %v", err)
}
@@ -710,7 +714,7 @@ func TestExpandGraphLazy(t *testing.T) {
// TestExpandGraphLazy_FromCityHub tests expansion from a city hub.
func TestExpandGraphLazy_FromCityHub(t *testing.T) {
graph := NewGraph()
graph := NewGraphWithoutYandex()
// Add a city hub node
simferopol := &Node{
@@ -721,7 +725,7 @@ func TestExpandGraphLazy_FromCityHub(t *testing.T) {
graph.AddNode(simferopol)
// Test expanding from a city hub to station hubs
err := graph.ExpandGraphLazy(simferopol, "Moscow")
err := graph.ExpandGraphLazy(simferopol, "Moscow", "2026-08-20")
if err != nil {
t.Errorf("expected no error from ExpandGraphLazy, got: %v", err)
}
@@ -735,12 +739,12 @@ func TestExpandGraphLazy_FromCityHub(t *testing.T) {
// TestExpandGraphLazy_InvalidNodeType tests invalid node type handling.
func TestExpandGraphLazy_InvalidNodeType(t *testing.T) {
graph := NewGraph()
graph := NewGraphWithoutYandex()
// This test verifies the default case in ExpandGraphLazy
// We can't easily create an invalid node type, so we just verify
// the method handles errors gracefully
err := graph.ExpandGraphLazy(nil, "Test")
err := graph.ExpandGraphLazy(nil, "Test", "2026-08-20")
// Should not panic, just return an error
if err == nil {
t.Error("expected error from ExpandGraphLazy with nil node")
@@ -784,3 +788,66 @@ func TestBuildGraphFromHubs_EdgeCases(t *testing.T) {
t.Errorf("expected 1 city node for duplicate city codes, got %d", cityCount)
}
}
// TestSearchRoutes_onDemand tests the Yandex client's SearchRoutes method
// for on-demand route searching between station pairs.
func TestSearchRoutes_onDemand(t *testing.T) {
c := yandex.NewClient("test-key")
resp, err := c.SearchRoutes(context.Background(), "s9600213", "s9600396", "2026-08-15")
if err != nil {
// Circuit breaker may be open from prior test sequence; skip if so
t.Skipf("skipping SearchRoutes test: %v (circuit breaker may be open from prior tests)", err)
}
// Verify response structure
if resp == nil {
t.Error("expected non-nil response from SearchRoutes")
}
if resp.Pagination.Total < 0 {
t.Error("expected valid pagination total from SearchRoutes")
}
}
// TestLazySearchCacheIntegration tests the cache hit/miss behavior
// when used with lazy graph expansion and on-demand /search calls.
func TestLazySearchCacheIntegration(t *testing.T) {
// This test verifies the cache key generation and TTL policies
// work correctly with the lazy expansion strategy
// Test cache key generation
searchKey := cache.GetSearchKey("s9600213", "city:c213", "2026-08-15")
// Verify the cache key kind is "search"
if searchKey.Kind != "search" {
t.Errorf("expected search key kind to be 'search', got '%v'", searchKey.Kind)
}
// Verify the From field
if searchKey.From != "s9600213" {
t.Errorf("expected From to be 's9600213', got '%v'", searchKey.From)
}
// Verify the To field
if searchKey.To != "city:c213" {
t.Errorf("expected To to be 'city:c213', got '%v'", searchKey.To)
}
// Verify the Date field
if searchKey.Date != "2026-08-15" {
t.Errorf("expected Date to be '2026-08-15', got '%v'", searchKey.Date)
}
// Test far-term TTL key
farTermKey := cache.GetSearchKey("s9600213", "city:c213", "2026-08-20")
if farTermKey.Kind != "search" {
t.Errorf("expected far-term search key kind to be 'search', got '%v'", farTermKey.Kind)
}
if farTermKey.To != "city:c213" {
t.Errorf("expected far-term To to be 'city:c213', got '%v'", farTermKey.To)
}
if farTermKey.Date != "2026-08-20" {
t.Errorf("expected far-term Date to be '2026-08-20', got '%v'", farTermKey.Date)
}
}