Change the behavior the Octal

- adds coverage
- change argument types
- use custom implementation for Base-8, Base-16

Author: dipu-bd

.github/workflows/test.yml

@@ -1,31 +1,61 @@
-name: test
+name: Test
 
 on:
   workflow_dispatch:
   push:
     branches: [master]
-    paths: ["**.dart", "**.yaml"]
+    paths: ['**.dart', '**.yaml', '**.yml']
   pull_request:
     branches: [master]
-    paths: ["**.dart", "**.yaml"]
-  schedule:
-    - cron: "0 0 * * 5" # m h d M w
+    paths: ['**.dart', '**.yaml', '**.yml']
 
 jobs:
-  build:
+  coverage:
     if: github.repository == 'bitanon/hashlib_codecs'
-    runs-on: ${{ matrix.os }}
     strategy:
       matrix:
-        os: [ubuntu-latest] # macos-latest, windows-latest
-        sdk: ["2.14.0", "2.19.6", "stable", "dev"]
+        os: [ubuntu-latest]
+        sdk: ['stable']
+    runs-on: ${{ matrix.os }}
+    steps:
+      - uses: actions/checkout@v4
+
+      - uses: dart-lang/setup-dart@v1
+        with:
+          sdk: ${{ matrix.sdk }}
+
+      - name: 'Install global packages'
+        run: |
+          dart pub global activate coverage
+          dart pub global activate cobertura
+          dart pub global activate junitreport
+
+      - name: Run tests with coverage
+        run: bash ./scripts/coverage.sh
+
+      - name: Upload results to Codecov
+        uses: codecov/codecov-action@v4
+        with:
+          token: ${{ secrets.CODECOV_TOKEN }}
+          file: ./coverage/cobertura.xml
 
+      - name: Upload test results to Codecov
+        if: ${{ !cancelled() }}
+        uses: codecov/test-results-action@v1
+        with:
+          token: ${{ secrets.CODECOV_TOKEN }}
+          file: ./coverage/junit.xml
+
+  test-on-stable:
+    strategy:
+      matrix:
+        os: [ubuntu-latest, macos-latest, windows-latest]
+        sdk: ['stable']
+    runs-on: ${{ matrix.os }}
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v4
 
-      # You can specify other versions if desired, see documentation here:
-      # https://github.com/dart-lang/setup-dart/blob/main/README.md
-      - uses: dart-lang/setup-dart@v1.3
+      - uses: dart-lang/setup-dart@v1
         with:
           sdk: ${{ matrix.sdk }}
 
@@ -38,12 +68,46 @@ jobs:
       - name: Analyze project source
         run: dart analyze --fatal-infos
 
-      - if: matrix.sdk != '2.14.0'
-        name: Generate documentation
-        run: dart doc --validate-links
+      - name: Run tests
+        run: dart test
+
+  test-on-legacy:
+    strategy:
+      matrix:
+        os: [ubuntu-latest, windows-latest]
+        sdk: ['2.14']
+    runs-on: ${{ matrix.os }}
+    steps:
+      - uses: actions/checkout@v4
+
+      - uses: dart-lang/setup-dart@v1
+        with:
+          sdk: ${{ matrix.sdk }}
+
+      - name: Run tests
+        run: dart test
+
+  test-on-beta:
+    strategy:
+      matrix:
+        os: [ubuntu-latest]
+        sdk: ['beta']
+    runs-on: ${{ matrix.os }}
+    steps:
+      - uses: actions/checkout@v4
+
+      - uses: dart-lang/setup-dart@v1
+        with:
+          sdk: ${{ matrix.sdk }}
+
+      - name: Install dependencies
+        run: dart pub get
 
-      - name: Run tests on vm
-        run: dart test -p vm
+      - name: Verify formatting
+        run: dart format --output=none --set-exit-if-changed .
+
+      - name: Analyze project source
+        run: dart analyze --fatal-infos
 
-      - name: Run tests on node
-        run: dart test -p node
+      - name: Run tests
+        run: dart test

.gitignore

@@ -3,9 +3,10 @@
 .dart_tool/
 .packages
 
-# Conventional directory for build outputs.
+# Conventional directory for outputs.
 build/
 doc/
+coverage/
 benchmark/**/*.exe
 test/**/*.exe
 

CHANGELOG.md

@@ -1,3 +1,9 @@
+# 2.6.0
+
+- [**Breaking Changes**]
+  - Change the behavior of the Base-8 encoder to follow the standard
+  - Accept only `List<int>` instead of `Iterable<int>` in converters
+
 # 2.5.0
 
 - Support UTF-8 encoding and decoding.

README.md

@@ -1,11 +1,13 @@
 # Hashlib Codecs
 
 [![plugin version](https://img.shields.io/pub/v/hashlib_codecs?label=pub)](https://pub.dev/packages/hashlib_codecs)
-[![dependencies](https://img.shields.io/badge/dependencies-zero-889)](https://github.com/bitanon/hashlib_codecs/blob/master/pubspec.yaml)
-[![dart support](https://img.shields.io/badge/dart-%3e%3d%202.14.0-39f?logo=dart)](https://dart.dev/guides/whats-new#september-8-2021-214-release)
+[![test](https://github.com/bitanon/hashlib_codecs/actions/workflows/test.yml/badge.svg?branch=master)](https://github.com/bitanon/hashlib_codecs/actions/workflows/test.yml)
+[![codecov](https://codecov.io/gh/bitanon/hashlib_codecs/graph/badge.svg?token=ISIYJ8MNI0)](https://codecov.io/gh/bitanon/hashlib_codecs)
 [![likes](https://img.shields.io/pub/likes/hashlib_codecs?logo=dart)](https://pub.dev/packages/hashlib_codecs/score)
 [![pub points](https://img.shields.io/pub/points/hashlib_codecs?logo=dart&color=teal)](https://pub.dev/packages/hashlib_codecs/score)
 [![popularity](https://img.shields.io/pub/popularity/hashlib_codecs?logo=dart)](https://pub.dev/packages/hashlib_codecs/score)
+[![dart support](https://img.shields.io/badge/dart-%3e%3d%202.14.0-39f?logo=dart)](https://dart.dev/guides/whats-new#september-8-2021-214-release)
+[![dependencies](https://img.shields.io/badge/dependencies-zero-889)](https://github.com/bitanon/hashlib_codecs/blob/master/pubspec.yaml)
 
 <!-- [![test](https://github.com/bitanon/hashlib_codecs/actions/workflows/test.yml/badge.svg)](https://github.com/bitanon/hashlib_codecs/actions/workflows/test.yml) -->
 

dart_test.yaml

@@ -5,7 +5,8 @@ concurrency: 8
 platforms: [vm, node]
 
 tags:
-  skip-js:
+  vm-only:
+    skip: true
     on_platform:
-      node:
-        skip: true
+      vm:
+        skip: false

lib/src/base16.dart

@@ -23,7 +23,7 @@ Base16Codec _codecFromParameters({
 /// - [codec] is the [Base16Codec] to use. It is derived from the other
 ///   parameters if not provided.
 String toHex(
-  Iterable<int> input, {
+  List<int> input, {
   Base16Codec? codec,
   bool upper = false,
 }) {
@@ -49,6 +49,5 @@ Uint8List fromHex(
   Base16Codec? codec,
 }) {
   codec ??= _codecFromParameters();
-  var out = codec.decoder.convert(input.codeUnits);
-  return Uint8List.fromList(out as List<int>);
+  return codec.decoder.convert(input.codeUnits);
 }

lib/src/base2.dart

@@ -17,7 +17,7 @@ import 'codecs/base2.dart';
 /// toBigInt(input).toRadixString(2)
 /// ```
 String toBinary(
-  Iterable<int> input, {
+  List<int> input, {
   Base2Codec codec = Base2Codec.standard,
 }) {
   var out = codec.encoder.convert(input);
@@ -44,6 +44,5 @@ Uint8List fromBinary(
   String input, {
   Base2Codec codec = Base2Codec.standard,
 }) {
-  var out = codec.decoder.convert(input.codeUnits);
-  return Uint8List.fromList(out as List<int>);
+  return codec.decoder.convert(input.codeUnits);
 }

lib/src/base32.dart

@@ -38,7 +38,7 @@ const _codecsWithPadding = {
 /// - [codec] is the [Base32Codec] to use. It is derived from the other
 ///   parameters if not provided.
 String toBase32(
-  Iterable<int> input, {
+  List<int> input, {
   Base32Codec? codec,
   bool lower = false,
   bool padding = true,
@@ -47,7 +47,7 @@ String toBase32(
     lower: lower,
     padding: padding,
   );
-  var out = codec.encoder.convert(input);
+  Iterable<int> out = codec.encoder.convert(input);
   if (!padding && _codecsWithPadding.contains(codec)) {
     out = out.takeWhile((x) => x != codec!.encoder.padding);
   }
@@ -76,5 +76,5 @@ Uint8List fromBase32(
 }) {
   codec ??= _codecFromParameters(padding: padding);
   var out = codec.decoder.convert(input.codeUnits);
-  return Uint8List.fromList(out as List<int>);
+  return Uint8List.fromList(out);
 }

lib/src/base64.dart

@@ -34,7 +34,7 @@ const _codecsWithPadding = {
 /// - [codec] is the [Base64Codec] to use. It is derived from the other
 ///   parameters if not provided.
 String toBase64(
-  Iterable<int> input, {
+  List<int> input, {
   Base64Codec? codec,
   bool url = false,
   bool padding = true,
@@ -43,7 +43,7 @@ String toBase64(
     url: url,
     padding: padding,
   );
-  var out = codec.encoder.convert(input);
+  Iterable<int> out = codec.encoder.convert(input);
   if (!padding && _codecsWithPadding.contains(codec)) {
     out = out.takeWhile((x) => x != codec!.encoder.padding);
   }
@@ -73,5 +73,5 @@ Uint8List fromBase64(
 }) {
   codec ??= _codecFromParameters(padding: padding);
   var out = codec.decoder.convert(input.codeUnits);
-  return Uint8List.fromList(out as List<int>);
+  return Uint8List.fromList(out);
 }

lib/src/base8.dart

@@ -17,7 +17,7 @@ import 'codecs/base8.dart';
 /// toBigInt(input).toRadixString(8)
 /// ```
 String toOctal(
-  Iterable<int> input, {
+  List<int> input, {
   Base8Codec codec = Base8Codec.standard,
 }) {
   var out = codec.encoder.convert(input);
@@ -44,6 +44,5 @@ Uint8List fromOctal(
   String input, {
   Base8Codec codec = Base8Codec.standard,
 }) {
-  var out = codec.decoder.convert(input.codeUnits);
-  return Uint8List.fromList(out as List<int>);
+  return codec.decoder.convert(input.codeUnits);
 }