Solved.

This is correct according to PEP 8 styling for python code but in terms of 1 liners, hardly most of them follow this structure. I guess it's all based on preference.

According to PEP 8 – Style Guide for Python Code

Imports are always put at the top of the file, just after any module comments and docstrings, and before module globals and constants

Additionally,

Compound statements (multiple statements on the same line) are generally discouraged:

Correct:

if foo == 'blah':
    do_blah_thing()
do_one()
do_two()
do_three()

Rather not:

# Wrong:
if foo == 'blah': do_blah_thing()
do_one(); do_two(); do_three()

I don't think it matters if you use an import outside or inside since the code is still one line right?

Same here.

Hi,

I cannot, because kata are almost unsearchable in the DB...
If you want to take your chances, just close the issue, but don't get surprized if the kata gets retired, or if someone actually finds the duplicate later and makes your kata unapprovable again.

Cheers

I think the test cases are weak, the extra notes at the bottom of the description are not tested at all. (I passed without considering self loops)

As always

Yes, those all involve binary search trees, but none of them, as far as I can tell, actually ask you to implement a search through a binary search tree. Can you point to one of these that actually asks the same thing I am?

Hi,

That's a duplicate (peek most of those). You shouldn't republish it. You still can fix it if you want, to get feedback about the kata creation process.

Cheers

The default Python code has a tab before return True. Codewars' editor defaults to spaces, so keeping the supplied return statement while adding any other edits results in a inconsistent use of tabs and spaces error.

Okay, I have rewritten it to follow common Java practices, accessing fields through getters.

I suppose I can include that if I make some very large random tests that will time out with an O(n) algorithm but not with an O(log n) algorithm.

Sorry, Codewars doesn't seem to have a way to save drafts. Should be fixed now.

src/main/java/Node.java:18: error: ',', ')', or '[' expected
  public Node(int value, Node left = null, Node right = null) {

Error when running tests.