twilio/config_test.go at main · OneBusAway/twilio · GitHub

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
package main

import (
	"os"
	"testing"

	"oba-twilio/client"

	"github.com/stretchr/testify/assert"
)

func TestServerConfiguration(t *testing.T) {
	tests := []struct {
		name        string
		envURL      string
		expectedURL string
	}{
		{
			name:        "Default Puget Sound",
			envURL:      "",
			expectedURL: "https://api.pugetsound.onebusaway.org",
		},
		{
			name:        "Tampa server",
			envURL:      "https://api.tampa.onebusaway.org",
			expectedURL: "https://api.tampa.onebusaway.org",
		},
		{
			name:        "Unitrans server",
			envURL:      "https://api.unitrans.onebusawaycloud.com",
			expectedURL: "https://api.unitrans.onebusawaycloud.com",
		},
		{
			name:        "Local development",
			envURL:      "http://localhost:8080",
			expectedURL: "http://localhost:8080",
		},
	}

	for _, tt := range tests {
		t.Run(tt.name, func(t *testing.T) {
			originalEnv := os.Getenv("ONEBUSAWAY_BASE_URL")
			defer func() {
				if err := os.Setenv("ONEBUSAWAY_BASE_URL", originalEnv); err != nil {
					t.Errorf("Failed to restore env var: %v", err)
				}
			}()

			if tt.envURL != "" {
				if err := os.Setenv("ONEBUSAWAY_BASE_URL", tt.envURL); err != nil {
					t.Fatalf("Failed to set env var: %v", err)
				}
			} else {
				if err := os.Unsetenv("ONEBUSAWAY_BASE_URL"); err != nil {
					t.Fatalf("Failed to unset env var: %v", err)
				}
			}

			obaBaseURL := os.Getenv("ONEBUSAWAY_BASE_URL")
			if obaBaseURL == "" {
				obaBaseURL = "https://api.pugetsound.onebusaway.org"
			}

			assert.Equal(t, tt.expectedURL, obaBaseURL)

			obaClient := client.NewOneBusAwayClient(obaBaseURL, "test-key")
			assert.Equal(t, tt.expectedURL, obaClient.BaseURL)
		})
	}
}

func TestAPIKeyConfiguration(t *testing.T) {
	tests := []struct {
		name        string
		envKey      string
		expectedKey string
	}{
		{
			name:        "Valid API key",
			envKey:      "valid-api-key-123",
			expectedKey: "valid-api-key-123",
		},
		{
			name:        "OneBusAway API key",
			envKey:      "org.onebusaway.iphone",
			expectedKey: "org.onebusaway.iphone",
		},
	}

	for _, tt := range tests {
		t.Run(tt.name, func(t *testing.T) {
			originalEnv := os.Getenv("ONEBUSAWAY_API_KEY")
			defer func() {
				if err := os.Setenv("ONEBUSAWAY_API_KEY", originalEnv); err != nil {
					t.Errorf("Failed to restore env var: %v", err)
				}
			}()

			if err := os.Setenv("ONEBUSAWAY_API_KEY", tt.envKey); err != nil {
				t.Fatalf("Failed to set env var: %v", err)
			}

			obaAPIKey := os.Getenv("ONEBUSAWAY_API_KEY")
			// In tests, we skip the validation logic since we're testing client construction

			assert.Equal(t, tt.expectedKey, obaAPIKey)

			obaClient := client.NewOneBusAwayClient("https://test.com", obaAPIKey)
			assert.Equal(t, tt.expectedKey, obaClient.APIKey)
		})
	}
}

func TestAPIKeyValidation(t *testing.T) {
	tests := []struct {
		name       string
		apiKey     string
		shouldFail bool
	}{
		{"Empty key should fail", "", true},
		{"Test key should pass", "test", false},
		{"TEST key should pass", "TEST", false},
		// "placeholder" was rejected before the issue #11 fix; any non-empty
		// value is now accepted, so it must pass.
		{"Placeholder should now pass", "placeholder", false},
		{"Valid key should pass", "valid-api-key", false},
		{"OneBusAway key should pass", "org.onebusaway.iphone", false},
	}

	for _, tt := range tests {
		t.Run(tt.name, func(t *testing.T) {
			err := validateAPIKey(tt.apiKey)
			if tt.shouldFail {
				assert.ErrorContains(t, err, "ONEBUSAWAY_API_KEY")
			} else {
				assert.NoError(t, err)
			}
		})
	}
}