coder
diff --git a/‎coderd/database/dbmem/dbmem.go
Lines changed: 3 additions & 3 deletions b/‎coderd/database/dbmem/dbmem.go
Lines changed: 3 additions & 3 deletions
diff --git a/‎coderd/database/dump.sql
Lines changed: 1 addition & 1 deletion b/‎coderd/database/dump.sql
Lines changed: 1 addition & 1 deletion
diff --git a/‎coderd/database/migrations/000301_system_user.up.sql
Lines changed: 1 addition & 1 deletion b/‎coderd/database/migrations/000301_system_user.up.sql
Lines changed: 1 addition & 1 deletion
diff --git a/‎coderd/database/models.go
Lines changed: 1 addition & 1 deletion b/‎coderd/database/models.go
Lines changed: 1 addition & 1 deletion
diff --git a/‎coderd/database/queries.sql.go
Lines changed: 1 addition & 1 deletion b/‎coderd/database/queries.sql.go
Lines changed: 1 addition & 1 deletion
diff --git a/‎coderd/users_test.go
Lines changed: 4 additions & 4 deletions b/‎coderd/users_test.go
Lines changed: 4 additions & 4 deletions
@@ -1555,7 +1555,7 @@ func (q *FakeQuerier) AllUserIDs(_ context.Context, includeSystem bool) ([]uuid.
 	defer q.mutex.RUnlock()
 	userIDs := make([]uuid.UUID, 0, len(q.users))
 	for idx := range q.users {
-		if !includeSystem && q.users[idx].IsSystem.Valid && q.users[idx].IsSystem.Bool {
+		if !includeSystem && q.users[idx].IsSystem {
 			continue
 		}
 
@@ -2656,7 +2656,7 @@ func (q *FakeQuerier) GetActiveUserCount(_ context.Context, includeSystem bool)
 
 	active := int64(0)
 	for _, u := range q.users {
-		if !includeSystem && u.IsSystem.Valid && u.IsSystem.Bool {
+		if !includeSystem && u.IsSystem {
 			continue
 		}
 
@@ -6221,7 +6221,7 @@ func (q *FakeQuerier) GetUserCount(_ context.Context, includeSystem bool) (int64
 			existing++
 		}
 
-		if !includeSystem && u.IsSystem.Valid && u.IsSystem.Bool {
+		if !includeSystem && u.IsSystem {
 			continue
 		}
 	}
 
@@ -1,5 +1,5 @@
 ALTER TABLE users
-	ADD COLUMN is_system bool DEFAULT false;
+	ADD COLUMN is_system bool DEFAULT false NOT NULL;
 
 CREATE INDEX user_is_system_idx ON users USING btree (is_system);
 
 
@@ -2450,7 +2450,7 @@ func TestSystemUserBehaviour(t *testing.T) {
 
 	var systemUser, regularUser database.GetUsersRow
 	for _, u := range users {
-		if u.IsSystem.Bool {
+		if u.IsSystem {
 			systemUser = u
 		} else {
 			regularUser = u
@@ -2459,9 +2459,9 @@ func TestSystemUserBehaviour(t *testing.T) {
 	require.NotNil(t, systemUser)
 	require.NotNil(t, regularUser)
 
-	require.True(t, systemUser.IsSystem.Bool)
+	require.True(t, systemUser.IsSystem)
 	require.Equal(t, systemUser.ID, prebuilds.SystemUserID)
-	require.False(t, regularUser.IsSystem.Bool)
+	require.False(t, regularUser.IsSystem)
 	require.Equal(t, regularUser.ID, other.ID)
 
 	// =================================================================================================================
@@ -2474,7 +2474,7 @@ func TestSystemUserBehaviour(t *testing.T) {
 	// Then: only regular users are returned.
 	require.NoError(t, err)
 	require.Len(t, users, 1)
-	require.False(t, users[0].IsSystem.Bool)
+	require.False(t, users[0].IsSystem)
 
 	// =================================================================================================================
Original file line number	Diff line number	Diff line change
`@@ -1555,7 +1555,7 @@ func (q *FakeQuerier) AllUserIDs(_ context.Context, includeSystem bool) ([]uuid.`
`1555`	`1555`	`defer q.mutex.RUnlock()`
`1556`	`1556`	`userIDs := make([]uuid.UUID, 0, len(q.users))`
`1557`	`1557`	`for idx := range q.users {`
`1558`		`- if !includeSystem && q.users[idx].IsSystem.Valid && q.users[idx].IsSystem.Bool {`
	`1558`	`+ if !includeSystem && q.users[idx].IsSystem {`
`1559`	`1559`	`continue`
`1560`	`1560`	`}`
`1561`	`1561`
`@@ -2656,7 +2656,7 @@ func (q *FakeQuerier) GetActiveUserCount(_ context.Context, includeSystem bool)`
`2656`	`2656`
`2657`	`2657`	`active := int64(0)`
`2658`	`2658`	`for _, u := range q.users {`
`2659`		`- if !includeSystem && u.IsSystem.Valid && u.IsSystem.Bool {`
	`2659`	`+ if !includeSystem && u.IsSystem {`
`2660`	`2660`	`continue`
`2661`	`2661`	`}`
`2662`	`2662`
`@@ -6221,7 +6221,7 @@ func (q *FakeQuerier) GetUserCount(_ context.Context, includeSystem bool) (int64`
`6221`	`6221`	`existing++`
`6222`	`6222`	`}`
`6223`	`6223`
`6224`		`- if !includeSystem && u.IsSystem.Valid && u.IsSystem.Bool {`
	`6224`	`+ if !includeSystem && u.IsSystem {`
`6225`	`6225`	`continue`
`6226`	`6226`	`}`
`6227`	`6227`	`}`