ahmadsamir created this revision.
ahmadsamir added reviewers: Frameworks, dfaure, sitter, meven.
Herald added a project: Frameworks.
ahmadsamir requested review of this revision.

REVISION SUMMARY
  Extend krununittest to test ktelnetservice.
  
  Use CMake target_compile_definitions() to set the CMAKE* vars that need
  to be accessed from the code in krununittest.

TEST PLAN
  $ kde-open5 ssh://user@1.1.1.1
  
    outputs:
    command= "ktelnetservice5 %u" args= ("ktelnetservice5 %u", 
"ssh://user@1.1.1.1")
    kf5.kio.core: couldn't create slave: "klauncher said: Unknown protocol 
'ssh'.\n"
  
  and you get a message box with "File not found: ssh://user@1.1.1.1".
  
  Apply diff then try again, now a terminal window is launched by
  ktelnetservice5 and ssh is invoked as expected.
  
  BUG: 418258
  
  FIXED-IN: 5.69

REPOSITORY
  R241 KIO

BRANCH
  l-krun-ssh (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D27999

AFFECTED FILES
  autotests/CMakeLists.txt
  autotests/krununittest.cpp
  autotests/krununittest.h
  src/core/desktopexecparser.cpp

To: ahmadsamir, #frameworks, dfaure, sitter, meven
Cc: kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns

Reply via email to