Skip to content

Conversation

@opatry
Copy link
Owner

@opatry opatry commented May 24, 2025

Description

  • Cmd+Shift+N to create new task list
  • Cmd+N to create new task

UI test tentative doesn't work, most likely due to menu node not being part of Compose scene.

@Test
fun `when clicking on show network logs then should trigger callback`() = runComposeUiTest {
  var isNetworkLogClicked = false
  setContent {
      Window(
          onCloseRequest = {},
          title = "Test",
      ) {
          AppMenuBar(
              showDevelopmentTools = true,
              onNewTaskListClick = {},
              onNewTaskClick = {},
              onAboutClick = {},
              onNetworkLogClick = { isNetworkLogClicked = true },
          )
      }
  }

  onNodeWithText("Tools")
      .assertIsDisplayed()
      .performClick()

  onNodeWithText("Show network logs")
      .assertIsDisplayed()
      .performClick()
}

Checklist

  • I have read the CONTRIBUTING guide
  • Code compiles correctly
  • Created tests which fail without the change (if possible)
  • All tests passing

@opatry opatry linked an issue May 24, 2025 that may be closed by this pull request
compose.resources {
publicResClass = false
packageOfResClass = "net.opatry.tasks.app.resources"
generateResClass = always
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

auto doesn't work, don't know why…

@opatry opatry merged commit bb8da0b into main May 24, 2025
2 checks passed
@opatry opatry deleted the desktop-app-menu branch May 24, 2025 14:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Use keyboard shortcuts on desktop (and menu?)

2 participants