Had a logical thinking test (15 minutes, with simple figures, nothing too complicated), followed by a talk with one of their Talent Acquisition managers, who was quite friendly.
Then I had the technical challenge - it was hosted on Qualified -. My code passed all the tests, and with considerably good performance, so my surprise was major when I got their feedback. I'll copy and paste because it's worth noting their phrasing:
Solution 1
"return statement holds too many operations, making it hard to read and maintain." <-- adding three strings is too many operations now
Solution 2:
"elegant solution, but usage of let for values that don't change" <-- sorry I forgot to make a const of something that will be converted into a var eventually down the line
Solution 3
"at a certain point there are 5 levels of nesting in this code, making it hard to maintain and overly complex" <-- if/else statements
"too many comments for obvious things makes things messy" <-- they ask to explain the code
So basically they penalize the kind of things any proper IDE would have automatically corrected, using the else part of an if statement - is that really messier? - and comments on code that, at the same time, they ask you to explain.
Pretty bad and disappointing experience after all, since they obviously care more about random style choices than about intelligence. I had heard some good things the company, and really bad things about their hiring process. At least now I know the bad things were true.