Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 0 additions & 1 deletion testing/dart/assets_test.dart
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@

import 'dart:async';
import 'dart:convert';
import 'dart:io';
import 'dart:typed_data';
import 'dart:ui';

Expand Down
2 changes: 1 addition & 1 deletion testing/dart/codec_test.dart
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,7 @@ void main() {
});

test('disposed decoded image', () async {
Uint8List data = await _getSkiaResource('flutter_logo.jpg').readAsBytes();
final Uint8List data = await _getSkiaResource('flutter_logo.jpg').readAsBytes();
final ui.Codec codec = await ui.instantiateImageCodec(data);
final ui.FrameInfo frameInfo = await codec.getNextFrame();
expect(frameInfo.image, isNotNull);
Expand Down
2 changes: 1 addition & 1 deletion testing/dart/compositing_test.dart
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.

import 'dart:typed_data' show Float64List, Uint32List, ByteData;
import 'dart:typed_data' show Float64List, ByteData;
import 'dart:ui';

import 'package:litetest/litetest.dart';
Expand Down
1 change: 0 additions & 1 deletion testing/dart/image_shader_test.dart
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.

import 'dart:async';
import 'dart:typed_data';
import 'dart:ui';

Expand Down
30 changes: 15 additions & 15 deletions testing/dart/paragraph_test.dart
Original file line number Diff line number Diff line change
Expand Up @@ -70,36 +70,36 @@ void main() {
));
builder.addText('Test Ahem');
final Paragraph paragraph = builder.build();
paragraph.layout(ParagraphConstraints(width: fontSize * 5.0));
paragraph.layout(const ParagraphConstraints(width: fontSize * 5.0));

// Wraps to two lines.
expect(paragraph.height, closeTo(fontSize * 2.0, 0.001));

final TextPosition wrapPositionDown = TextPosition(
const TextPosition wrapPositionDown = TextPosition(
offset: 5,
affinity: TextAffinity.downstream,
);
TextRange line = paragraph.getLineBoundary(wrapPositionDown);
expect(line.start, 5);
expect(line.end, 9);

final TextPosition wrapPositionUp = TextPosition(
const TextPosition wrapPositionUp = TextPosition(
offset: 5,
affinity: TextAffinity.upstream,
);
line = paragraph.getLineBoundary(wrapPositionUp);
expect(line.start, 0);
expect(line.end, 5);

final TextPosition wrapPositionStart = TextPosition(
const TextPosition wrapPositionStart = TextPosition(
offset: 0,
affinity: TextAffinity.downstream,
);
line = paragraph.getLineBoundary(wrapPositionStart);
expect(line.start, 0);
expect(line.end, 5);

final TextPosition wrapPositionEnd = TextPosition(
const TextPosition wrapPositionEnd = TextPosition(
offset: 9,
affinity: TextAffinity.downstream,
);
Expand All @@ -119,36 +119,36 @@ void main() {
));
builder.addText('القاهرةالقاهرة');
final Paragraph paragraph = builder.build();
paragraph.layout(ParagraphConstraints(width: fontSize * 5.0));
paragraph.layout(const ParagraphConstraints(width: fontSize * 5.0));

// Wraps to three lines.
expect(paragraph.height, closeTo(fontSize * 3.0, 0.001));

final TextPosition wrapPositionDown = TextPosition(
const TextPosition wrapPositionDown = TextPosition(
offset: 5,
affinity: TextAffinity.downstream,
);
TextRange line = paragraph.getLineBoundary(wrapPositionDown);
expect(line.start, 5);
expect(line.end, 10);

final TextPosition wrapPositionUp = TextPosition(
const TextPosition wrapPositionUp = TextPosition(
offset: 5,
affinity: TextAffinity.upstream,
);
line = paragraph.getLineBoundary(wrapPositionUp);
expect(line.start, 0);
expect(line.end, 5);

final TextPosition wrapPositionStart = TextPosition(
const TextPosition wrapPositionStart = TextPosition(
offset: 0,
affinity: TextAffinity.downstream,
);
line = paragraph.getLineBoundary(wrapPositionStart);
expect(line.start, 0);
expect(line.end, 5);

final TextPosition wrapPositionEnd = TextPosition(
const TextPosition wrapPositionEnd = TextPosition(
offset: 9,
affinity: TextAffinity.downstream,
);
Expand All @@ -168,12 +168,12 @@ void main() {
));
builder.addText('Test\n\nAhem');
final Paragraph paragraph = builder.build();
paragraph.layout(ParagraphConstraints(width: fontSize * 5.0));
paragraph.layout(const ParagraphConstraints(width: fontSize * 5.0));

// Three lines due to line breaks, with the middle line being empty.
expect(paragraph.height, closeTo(fontSize * 3.0, 0.001));

final TextPosition emptyLinePosition = TextPosition(
const TextPosition emptyLinePosition = TextPosition(
offset: 5,
affinity: TextAffinity.downstream,
);
Expand All @@ -182,23 +182,23 @@ void main() {
expect(line.end, 5);

// Since these are hard newlines, TextAffinity has no effect here.
final TextPosition emptyLinePositionUpstream = TextPosition(
const TextPosition emptyLinePositionUpstream = TextPosition(
offset: 5,
affinity: TextAffinity.upstream,
);
line = paragraph.getLineBoundary(emptyLinePositionUpstream);
expect(line.start, 5);
expect(line.end, 5);

final TextPosition endOfFirstLinePosition = TextPosition(
const TextPosition endOfFirstLinePosition = TextPosition(
offset: 4,
affinity: TextAffinity.downstream,
);
line = paragraph.getLineBoundary(endOfFirstLinePosition);
expect(line.start, 0);
expect(line.end, 4);

final TextPosition startOfLastLinePosition = TextPosition(
const TextPosition startOfLastLinePosition = TextPosition(
offset: 6,
affinity: TextAffinity.downstream,
);
Expand Down
9 changes: 7 additions & 2 deletions testing/dart/serial_gc_test.dart
Original file line number Diff line number Diff line change
Expand Up @@ -6,11 +6,16 @@

import 'package:litetest/litetest.dart';

int use(List<int> a) {
return a[0];
}

void main() {
test('Serial GC option test ', () async {
bool threw = false;
const bool threw = false;
for (int i = 0; i < 100; i++) {
var a = <int>[100];
final List<int> a = <int>[100];
use(a);
}
expect(threw, false);
});
Expand Down
6 changes: 3 additions & 3 deletions testing/dart/text_test.dart
Original file line number Diff line number Diff line change
Expand Up @@ -252,7 +252,7 @@ void testFontVariation() {
));
baseBuilder.addText('Hello');
final Paragraph baseParagraph = baseBuilder.build();
baseParagraph.layout(ParagraphConstraints(width: double.infinity));
baseParagraph.layout(const ParagraphConstraints(width: double.infinity));
final double baseWidth = baseParagraph.minIntrinsicWidth;

final ParagraphBuilder wideBuilder = ParagraphBuilder(ParagraphStyle(
Expand All @@ -262,11 +262,11 @@ void testFontVariation() {
wideBuilder.pushStyle(TextStyle(
fontFamily: 'RobotoSerif',
fontSize: 40.0,
fontVariations: <FontVariation>[FontVariation('wght', 900.0)]
fontVariations: <FontVariation>[const FontVariation('wght', 900.0)],
));
wideBuilder.addText('Hello');
final Paragraph wideParagraph = wideBuilder.build();
wideParagraph.layout(ParagraphConstraints(width: double.infinity));
wideParagraph.layout(const ParagraphConstraints(width: double.infinity));
final double wideWidth = wideParagraph.minIntrinsicWidth;

expect(wideWidth, greaterThan(baseWidth));
Expand Down