Function TestAddCopilotRequestsPermissionToContent

pkg/cli/add_command_test.go:681–712

(t *testing.T)

}

func TestAddCopilotRequestsPermissionToContent(t *testing.T) {
	t.Run("adds permission to workflow without existing permissions block", func(t *testing.T) {
		content := "---\nengine: copilot\n---\nDo the thing.\n"
		result, err := addCopilotRequestsPermissionToContent(content)
		require.NoError(t, err)
		assert.Contains(t, result, "permissions:")
		assert.Contains(t, result, "copilot-requests: write")
	})

	t.Run("adds permission to workflow with existing permissions block", func(t *testing.T) {
		content := "---\nengine: copilot\npermissions:\n  contents: read\n---\nDo the thing.\n"
		result, err := addCopilotRequestsPermissionToContent(content)
		require.NoError(t, err)
		assert.Contains(t, result, "copilot-requests: write")
		assert.Contains(t, result, "contents: read")
	})

	t.Run("is idempotent when permission already present", func(t *testing.T) {
		content := "---\nengine: copilot\npermissions:\n  copilot-requests: write\n---\nDo the thing.\n"
		result, err := addCopilotRequestsPermissionToContent(content)
		require.NoError(t, err)
		count := strings.Count(result, "copilot-requests: write")
		assert.Equal(t, 1, count, "copilot-requests: write should appear exactly once")
	})

	t.Run("returns error when permissions is a non-mapping scalar", func(t *testing.T) {
		content := "---\nengine: copilot\npermissions: read-all\n---\nDo the thing.\n"
		_, err := addCopilotRequestsPermissionToContent(content)
		require.Error(t, err)
		assert.Contains(t, err.Error(), "non-mapping scalar")
	})
}

func TestAddWorkflowWithTracking_CopilotRequestsPermission(t *testing.T) {
	t.Run("injects copilot-requests permission when option is set", func(t *testing.T) {