jpmschweitzerandClaude Opus 4.8
6d0ebab721
Raise the declared minimums in pubspec.yaml to what our deps already require: Flutter >=3.35.0 / Dart >=3.9.0 (was 3.19.0 / 3.5.0). alchemist 0.12 needs Flutter 3.32; Dart 3.9 first ships in Flutter 3.35, so 3.35 is the binding floor. Pin the exact build toolchain in .fvmrc (Flutter 3.44.1). Moving to the Dart 3.9 language level switches `dart format` to the new "tall" style and enables two new lints. This commit is the resulting mechanical churn, isolated from any behaviour change: - whole-tree `dart format` reformat (tall style) - `dart fix` for unnecessary_underscores + use_null_aware_elements No runtime behaviour change; `make test` green. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
56 lines
2.3 KiB
Dart
56 lines
2.3 KiB
Dart
/// Closeout tests for the last single-digit-uncovered lines in
|
|
/// `lib/src/terminal/`: pointer_input constructors, the abstract
|
|
/// TerminalMouseHandler constructor, the reflow padding branch in
|
|
/// Buffer.resize, and the wide-char skip branch in
|
|
/// TerminalPainter.paintLine.
|
|
library;
|
|
|
|
import 'dart:ui';
|
|
|
|
import 'package:clide/src/terminal/src/core/mouse/handler.dart';
|
|
import 'package:clide/src/terminal/src/terminal.dart';
|
|
import 'package:clide/src/terminal/src/ui/painter.dart';
|
|
import 'package:clide/src/terminal/src/ui/pointer_input.dart';
|
|
import 'package:clide/src/terminal/src/ui/terminal_text_style.dart';
|
|
import 'package:clide/src/terminal/src/ui/themes.dart';
|
|
import 'package:flutter/painting.dart';
|
|
import 'package:test/test.dart';
|
|
|
|
class _StubMouseHandler extends TerminalMouseHandler {
|
|
const _StubMouseHandler();
|
|
@override
|
|
String? call(TerminalMouseEvent event) => null;
|
|
}
|
|
|
|
void main() {
|
|
test('PointerInputs.none / .all produce expected sets', () {
|
|
expect(const PointerInputs.none().inputs, isEmpty);
|
|
expect(const PointerInputs.all().inputs, containsAll(PointerInput.values));
|
|
});
|
|
|
|
test('TerminalMouseHandler abstract const constructor is reachable via a subclass', () {
|
|
const h = _StubMouseHandler();
|
|
expect(h, isA<TerminalMouseHandler>());
|
|
});
|
|
|
|
test('Buffer.resize pads reflow output up to the new height', () {
|
|
final t = Terminal(maxLines: 200, onOutput: (_) {});
|
|
// Setup: narrow viewport with wrapped continuation lines. Widening
|
|
// collapses them in reflow → output rows < newHeight → the
|
|
// pad-with-empty-lines branch in Buffer.resize runs.
|
|
t.resize(5, 30, 8, 16);
|
|
t.write('aaaaaaaaaaaaaaaaaaaa'); // wraps 4 times
|
|
t.resize(80, 30, 8, 16); // widen, same height
|
|
expect(t.buffer.lines.length, 30);
|
|
});
|
|
|
|
test('TerminalPainter.paintLine handles wide (CJK) cells without skipping the skip', () {
|
|
final t = Terminal(maxLines: 100, onOutput: (_) {});
|
|
// CJK ideograph — terminal stores it as a width-2 cell, so paintLine
|
|
// advances `i` twice and the wide-char branch fires.
|
|
t.write('漢字');
|
|
final p = TerminalPainter(theme: TerminalThemes.defaultTheme, textStyle: const TerminalStyle(), textScaler: const TextScaler.linear(1.0));
|
|
p.paintLine(Canvas(PictureRecorder()), Offset.zero, t.buffer.lines[0]);
|
|
});
|
|
}
|